Skip to content

Fix copy-pasted account wording in property access binding samples - #74

Open
Avicennasis wants to merge 1 commit into
googleanalytics:mainfrom
TypoScan3000:docs/fix-typos
Open

Avicennasis wants to merge 1 commit into
googleanalytics:mainfrom
TypoScan3000:docs/fix-typos

Conversation

@Avicennasis

Copy link
Copy Markdown

Summary

The six google-analytics-admin/properties_access_bindings_*.py samples were copied from the account access binding samples and still describe the ID as an account access binding ID. Fixed in properties_access_bindings_{get,update,delete,batch_get,batch_update,batch_delete}.py:

  • TODO comment: "account access binding ID" -> "property access binding ID"
  • docstring for property_access_binding_id: "Google Analytics account access binding ID." -> "Google Analytics property access binding ID."
  • placeholder value "YOUR-ACCOUNT-ACCESS-BINDING-ID" -> "YOUR-PROPERTY-ACCESS-BINDING-ID". This is the string users replace before running the sample; the tests read the ID from GA_TEST_PROPERTY_ACCESS_BINDING_ID and never touch it

No functional changes — comments, docstrings and a placeholder string only.

Signed-off-by: Léon Avic Simmons <Avicennasis@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant