-
Notifications
You must be signed in to change notification settings - Fork 436
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
Conversation
…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==
Here is the summary of changes. You are about to add 6 region tags.
This comment is generated by snippet-bot.
|
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.
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==
RunAccessReport
method (example: /v1alpha/accounts/1234567:runAccessReport)RunAccessReport
method (with bindings for account and property resources) to the Admin API v1beta
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.
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.
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 v1betaPiperOrigin-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 the
RunAccessReport
method (example: /v1alpha/accounts/1234567:runAccessReport)feat: add
GetEnhancedMeasurementSettings
,UpdateEnhancedMeasurementSettings
methods to the Admin API v1alphafeat: add
CreateConnectedSiteTag
,DeleteConnectedSiteTag
,ListConnectedSiteTags
methods to the Admin API v1alphafeat: add
EnhancedMeasurementSettings
,ConnectedSiteTag
resource types to the Admin API v1alphafeat: add
GetEnhancedMeasurementSettingsRequest
,UpdateEnhancedMeasurementSettingsRequest
,CreateConnectedSiteTagRequest
,CreateConnectedSiteTagResponse
,DeleteConnectedSiteTagRequest
,ListConnectedSiteTagsRequest
,ListConnectedSiteTagsResponse
feat: add
ENHANCED_MEASUREMENT_SETTINGS
option to theChangeHistoryResourceType
enumfeat: add
enhanced_measurement_settings
option to theChangeHistoryResource.resource
oneof fieldfeat: add
intraday_export_enabled
field to theBigQueryLink
resourcePiperOrigin-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