-
Notifications
You must be signed in to change notification settings - Fork 299
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
Add the new audience settings store. #10154
Comments
Hey @benbowler, the IB is looking good. A couple of points:
I'd suggest we take it further than this, and rename these where they are defined in
The IB seems to suggest adding these to I'd suggest creating a new file for these datastore additions, Also please consider if any additional actions would be needed e.g. I think we'll want |
IB ✅ |
QA Update ✅Performed smoke test on Audience Segmentation and it's looking good. Screen.Recording.2025-03-04.at.15.23.07.mov10154.-.AS.2.ok.mov |
Feature Description
Issue 2/3 to fix the bug #8888.
Add the new audience settings store, but don't integrate it into the Analytics store. Write tests just for the store slice.
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
Implementation Brief
Create isolated, tested, audiences settings store.
Note: this ticket should not integrate this settings store into the analytics module, this work will be done in #8888.
Update
assets/js/googlesitekit/datastore/user/audience-settings.js
:getAudienceSettings
togetUserAudienceSettings
saveAudienceSettings
tosaveUserAudienceSettings
resetAudienceSettings
toresetUserAudienceSettings
isSavingAudienceSettings
toisSavingUserAudienceSettings
Update the name of these selectors and actions everywhere they are used in the current codebase.
Create a new store file
assets/js/modules/analytics-4/datastore/audience-settings.js
:createFetchStore
:controlCallback
should useAPI.get
to send GET requests toGET:audience-settings
.getAvailableAudiences
*Note: the original, duplicate, definition ofgetAvailableAudiences
will be removed fromassets/js/modules/analytics-4/datastore/audiences.js
when this work is combined in Syncing audiences and custom dimensions in parallel can result in the cached audiences not being persisted #8888.getAudienceSegmentationSetupCompletedBy
createFetchStore
:controlCallback
should useAPI.set
to send POST requests toPOST:save-audience-settings
.setAvailableAudiences
,setAudienceSegmentationSetupCompletedBy
andsaveAudienceSettings
actions.Test Coverage
QA Brief
Changelog entry
The text was updated successfully, but these errors were encountered: