Automattic / Automattic/jetpack
WordAds: make ccpa consent shortcode AMP compatible
Open
[Feature] WordAds
[Pri] Low
AMP
Enhancement
- Dominant language
- PHP
- Stars
- 1.8k
- Forks
- 898
- Avg merge
- 1d 18h
- Merged PRs (30d)
- 774
Description
We introduced a new shortcode in #15260. It would be nice if it was compatible with AMP, maybe via the `amp-consent` element like we do for the Cookies & Consent widget:
https://github.com/Automattic/jetpack/blob/73d904f5f5d04d63cc828953d682075645c4d2ae/modules/widgets/eu-cookie-law/widget-amp.php#L9
https://amp.dev/documentation/components/amp-consent/
Contributor guide
Research direction
Start by reviewing the shortcode introduced in issue #15260, then compare the Cookies & Consent AMP implementation in modules/widgets/eu-cookie-law/widget-amp.php and the amp-consent documentation. Done means the WordAds CCPA consent shortcode works in AMP using the appropriate consent element.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- frontend, web-dev
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100