✨ [amp-consent] Add purposes to user actions for inline PromptUI#32470
✨ [amp-consent] Add purposes to user actions for inline PromptUI#32470micajuine-ho merged 11 commits intoampproject:masterfrom
Conversation
92a78c8 to
362041d
Compare
0c3450c to
0c86766
Compare
|
@dmanek PTAL Separated the "closing" actions from from the new Also moved the |
| dev().error(TAG, 'No consent state manager'); | ||
| return false; | ||
| } | ||
| return this.isPromptUIOn_; |
There was a problem hiding this comment.
nit: isPromptUiOn Sorry, missed this earlier.
There was a problem hiding this comment.
Yea it's weird, b/c in our config we have promptUI, promptUISrc, and postPromptUI... should get to changing this (maybe a fixit week item 👀)
e257493 to
b1d4938
Compare
Partial for #31607
setPurposeaction in the PromptUI, so that publishers can explicitly tell AMP which purposes shave been accepted and rejectedsetPurposeaccepts multiple args (purposes) and each must be set to True/False (denoting if the purpose has been consented to)The purposes will be stored in the
consentStateManager, which will eventually store these values in local storage (to cache) and propagate them to the consent policy manager (to block and unblock).Modified the existing
acceptandrejectactions in the PromptUI to accept apurposeConsentDefaultargument that will tell AMP to use this value as a fallback for all purposes state inpurposeConsentRequired.