Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: [AnalyticsAdmin] add RunAccessReport method (with bindings for account and property resources) to the Admin API v1beta #5985

Merged
merged 5 commits into from
Mar 25, 2023

Conversation

gcf-owl-bot[bot]
Copy link
Contributor

@gcf-owl-bot gcf-owl-bot bot commented Mar 20, 2023

  • Regenerate this pull request now.

feat: add AccessDimension, AccessMetric, AccessDateRange, AccessFilterExpression, AccessFilterExpressionList, AccessFilter, AccessStringFilter, AccessInListFilter, AccessNumericFilter, AccessBetweenFilter, NumericValue, AccessOrderBy, AccessDimensionHeader, AccessMetricHeader, AccessRow, AccessDimensionValue, AccessMetricValue, AccessQuota, AccessQuotaStatus types to the Admin API v1beta

PiperOrigin-RevId: 519058583

Source-Link: googleapis/googleapis@71bd02e

Source-Link: https://github.com/googleapis/googleapis-gen/commit/5d4f0f6b45923d62c7eec656bacc3f0633e011f7
Copy-Tag: eyJwIjoiQW5hbHl0aWNzQWRtaW4vLk93bEJvdC55YW1sIiwiaCI6IjVkNGYwZjZiNDU5MjNkNjJjN2VlYzY1NmJhY2MzZjA2MzNlMDExZjcifQ==

BEGIN_NESTED_COMMIT
feat: [AnalyticsAdmin] add account-level binding for theRunAccessReport method (example: /v1alpha/accounts/1234567:runAccessReport)
feat: add GetEnhancedMeasurementSettings, UpdateEnhancedMeasurementSettings methods to the Admin API v1alpha
feat: add CreateConnectedSiteTag, DeleteConnectedSiteTag, ListConnectedSiteTags methods to the Admin API v1alpha
feat: add EnhancedMeasurementSettings, ConnectedSiteTag resource types to the Admin API v1alpha
feat: add GetEnhancedMeasurementSettingsRequest,
UpdateEnhancedMeasurementSettingsRequest,CreateConnectedSiteTagRequest, CreateConnectedSiteTagResponse, DeleteConnectedSiteTagRequest, ListConnectedSiteTagsRequest, ListConnectedSiteTagsResponse
feat: add ENHANCED_MEASUREMENT_SETTINGS option to the ChangeHistoryResourceType enum
feat: add enhanced_measurement_settings option to the ChangeHistoryResource.resource oneof field
feat: add intraday_export_enabled field to the BigQueryLink resource

PiperOrigin-RevId: 518088512

Source-Link: googleapis/googleapis@061a9fb

Source-Link: https://github.com/googleapis/googleapis-gen/commit/c64b9cd38073b2202ce6eb07d58c6a8c581fce2c
Copy-Tag: eyJwIjoiQW5hbHl0aWNzQWRtaW4vLk93bEJvdC55YW1sIiwiaCI6ImM2NGI5Y2QzODA3M2IyMjAyY2U2ZWIwN2Q1OGM2YThjNTgxZmNlMmMifQ==
END_NESTED_COMMIT

BREAKING_CHANGE_REASON=This is an alpha API in a preview state, but also the class that changed hasn't been functional for two years

…ple: /v1alpha/accounts/1234567:runAccessReport)

feat: add `GetEnhancedMeasurementSettings`, `UpdateEnhancedMeasurementSettings` methods to the Admin API v1alpha
feat: add `CreateConnectedSiteTag`, `DeleteConnectedSiteTag`, `ListConnectedSiteTags` methods to the Admin API v1alpha
feat: add `EnhancedMeasurementSettings`, `ConnectedSiteTag` resource types to the Admin API v1alpha
feat: add `GetEnhancedMeasurementSettingsRequest`,
`UpdateEnhancedMeasurementSettingsRequest`,`CreateConnectedSiteTagRequest`, `CreateConnectedSiteTagResponse`, `DeleteConnectedSiteTagRequest`, `ListConnectedSiteTagsRequest`, `ListConnectedSiteTagsResponse`
feat: add `ENHANCED_MEASUREMENT_SETTINGS` option to the `ChangeHistoryResourceType` enum
feat: add `enhanced_measurement_settings` option to the `ChangeHistoryResource.resource` oneof field
feat: add `intraday_export_enabled` field to the `BigQueryLink` resource

PiperOrigin-RevId: 518088512

Source-Link: googleapis/googleapis@061a9fb

Source-Link: googleapis/googleapis-gen@c64b9cd
Copy-Tag: eyJwIjoiQW5hbHl0aWNzQWRtaW4vLk93bEJvdC55YW1sIiwiaCI6ImM2NGI5Y2QzODA3M2IyMjAyY2U2ZWIwN2Q1OGM2YThjNTgxZmNlMmMifQ==
@gcf-owl-bot gcf-owl-bot bot requested review from a team as code owners March 20, 2023 23:54
@snippet-bot
Copy link

snippet-bot bot commented Mar 20, 2023

Here is the summary of changes.

You are about to add 6 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@trusted-contributions-gcf trusted-contributions-gcf bot added the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 20, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 20, 2023
@trusted-contributions-gcf trusted-contributions-gcf bot added the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 20, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 20, 2023
Copy link
Contributor

@bshaffer bshaffer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The breaking changes are as follows:

Error: Method Google\Analytics\Admin\V1alpha\EnhancedMeasurementSettings#getPageViewsEnabled() was removed
Error: Method Google\Analytics\Admin\V1alpha\EnhancedMeasurementSettings#setPageViewsEnabled() was removed
Error: Method Google\Analytics\Admin\V1alpha\EnhancedMeasurementSettings#getPageLoadsEnabled() was removed
Error: Method Google\Analytics\Admin\V1alpha\EnhancedMeasurementSettings#setPageLoadsEnabled() was removed

I looked at the Piper RevID, and could not find any evidence of these fields being removed (and nothing is marked as breaking in the conventional commits). I'll do a little more digging to find out why this happened.

In the meantime, we can take this breaking change because 1. we are pre-GA (v0.10.0) and the API is still in alpha.

…perty resources) to the Admin API v1beta

feat: add `AccessDimension`, `AccessMetric`, `AccessDateRange`, `AccessFilterExpression`, `AccessFilterExpressionList`, `AccessFilter`, `AccessStringFilter`, `AccessInListFilter`, `AccessNumericFilter`, `AccessBetweenFilter`, `NumericValue`, `AccessOrderBy`, `AccessDimensionHeader`, `AccessMetricHeader`, `AccessRow`, `AccessDimensionValue`, `AccessMetricValue`, `AccessQuota`, `AccessQuotaStatus` types to the Admin API v1beta

PiperOrigin-RevId: 519058583

Source-Link: googleapis/googleapis@71bd02e

Source-Link: googleapis/googleapis-gen@5d4f0f6
Copy-Tag: eyJwIjoiQW5hbHl0aWNzQWRtaW4vLk93bEJvdC55YW1sIiwiaCI6IjVkNGYwZjZiNDU5MjNkNjJjN2VlYzY1NmJhY2MzZjA2MzNlMDExZjcifQ==
@gcf-owl-bot gcf-owl-bot bot changed the title feat: [AnalyticsAdmin] add account-level binding for theRunAccessReport method (example: /v1alpha/accounts/1234567:runAccessReport) feat: [AnalyticsAdmin] add RunAccessReport method (with bindings for account and property resources) to the Admin API v1beta Mar 24, 2023
@trusted-contributions-gcf trusted-contributions-gcf bot added the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 24, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 24, 2023
@trusted-contributions-gcf trusted-contributions-gcf bot added the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 24, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 24, 2023
Copy link
Contributor

@bshaffer bshaffer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay I found it. In this. commit, two years ago, the EnhancedMeasurementSettings class was removed, but as we don't track removals, the file stayed in the repo. Now, in this commit, which is the one that updated this PR, the class was added back, but without the two fields being complained about by our BC-checker.

@trusted-contributions-gcf trusted-contributions-gcf bot added the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 25, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Mar 25, 2023
@bshaffer bshaffer merged commit 9479f11 into main Mar 25, 2023
@bshaffer bshaffer deleted the owl-bot-copy-AnalyticsAdmin branch March 25, 2023 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant