-
Notifications
You must be signed in to change notification settings - Fork 8
docs: add samples for Measurement Protocol Secrets management methods #152
Conversation
# Conflicts: # samples/noxfile.py
… no longer supported in v1alpha
…properties.firebase_links tests as this is no longer supported in v1alpha
…rties.google_ads_links_list() test as the field has been renamed in v1alpha
# Conflicts: # samples/accounts_search_change_history_events_test.py
Here is the summary of changes. You are about to add 15 region tags.
This comment is generated by snippet-bot.
|
All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the ℹ️ Googlers: Go here for more info. |
All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the ℹ️ Googlers: Go here for more info. |
All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the ℹ️ Googlers: Go here for more info. |
2 similar comments
All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the ℹ️ Googlers: Go here for more info. |
All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the ℹ️ Googlers: Go here for more info. |
@googlebot I consent. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add observation in sample for deleting measurement protocol secrets, otherwise LGTM.
samples/properties_web_data_streams_measurement_protocol_secrets_delete.py
Outdated
Show resolved
Hide resolved
samples/properties_web_data_streams_measurement_protocol_secrets_delete.py
Outdated
Show resolved
Hide resolved
samples/properties_web_data_streams_measurement_protocol_secrets_delete.py
Outdated
Show resolved
Hide resolved
samples/properties_web_data_streams_measurement_protocol_secrets_delete.py
Outdated
Show resolved
Hide resolved
samples/properties_web_data_streams_measurement_protocol_secrets_delete.py
Outdated
Show resolved
Hide resolved
samples/properties_web_data_streams_measurement_protocol_secrets_delete.py
Outdated
Show resolved
Hide resolved
…ts_delete.py Co-authored-by: Anthonios Partheniou <partheniou@google.com>
…ts_delete.py Co-authored-by: Anthonios Partheniou <partheniou@google.com>
…ts_delete.py Co-authored-by: Anthonios Partheniou <partheniou@google.com>
…ts_delete.py Co-authored-by: Anthonios Partheniou <partheniou@google.com>
…ts_delete.py Co-authored-by: Anthonios Partheniou <partheniou@google.com>
…ts_delete.py Co-authored-by: Anthonios Partheniou <partheniou@google.com>
@parthea Thanks for spotting the mangled sample! |
🤖 I have created a release \*beep\* \*boop\* --- ### [0.7.1](https://www.github.com/googleapis/python-analytics-admin/compare/v0.7.0...v0.7.1) (2021-10-19) ### Documentation * **samples:** add samples for Measurement Protocol Secrets management methods ([#152](https://www.github.com/googleapis/python-analytics-admin/issues/152)) ([e264571](https://www.github.com/googleapis/python-analytics-admin/commit/e2645719d6fc518857f64482c48f60e1e0963fc7)) * **samples:** add samples for Conversion Event management methods ([#153](https://www.github.com/googleapis/python-analytics-admin/issues/153)) ([126f271](https://www.github.com/googleapis/python-analytics-admin/commit/126f2711c9fdedaa7cddfe8b3c7bdaff03d0297e)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
Fixes #<issue_number_goes_here> 🦕