-
Notifications
You must be signed in to change notification settings - Fork 153
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
Created a new page type for PNI Privacy Index page #11107
Created a new page type for PNI Privacy Index page #11107
Conversation
|
network-api/networkapi/templates/buyersguide/pages/consumer_creepometer_page_2023.html
Show resolved
Hide resolved
22c1b60
to
b13e304
Compare
network-api/networkapi/templates/buyersguide/pages/consumer_creepometer_page.html
Show resolved
Hide resolved
f444ec3
to
8cfdd26
Compare
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.
Great job on these, @mmmavis! I just have some minor comments
network-api/networkapi/wagtailpages/tests/buyersguide/test_consumer_creepometer_page.py
Show resolved
Hide resolved
network-api/networkapi/wagtailpages/tests/buyersguide/test_consumer_creepometer_page.py
Outdated
Show resolved
Hide resolved
network-api/networkapi/wagtailpages/tests/buyersguide/test_consumer_creepometer_page.py
Show resolved
Hide resolved
network-api/networkapi/wagtailpages/tests/buyersguide/test_consumer_creepometer_page.py
Outdated
Show resolved
Hide resolved
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.
LGTM 👍
* Created a new page type for PNI Privacy Index page (#11107) * Create a new page type for PNI annual consumer creep-o-meter * fix migration conflict * PNI Consumer Creep-O-Meter Page Hero (#11146) * got general section structure in * implemented hero section * PNI CC Page - Sub Nav + 3 middle sections (#11156) * implemented the "sub nav", "big picture trends", "best & worst products" and "by the numbers" sections * PNI CC Page - Quiz section (part 1) (#11211) * Quiz Section Part 1 work. Includes the product selection screen and the result screen only. * PNI CC Page - Outro section + Newsletter box at the bottom of Quiz section (#11219) * Modified the existing PNI newsletter box setup so it can be used for difference scenarios & built Outro section & added newsletter box to Quiz section * fix migration conflicts * linting * updated all face existing SVGs * PNI CC Page - remaining work for Quiz Part 2 (#11224) * Added the newsletter signup prompt and thank you messages to quiz flow * PNI CC Page - Animated Eyes (#11241) * moved hero section to its own fragment file * animated creepy eyes * 11213 quiz share buttons (#11239) * Share buttons on Quiz section * PNI CC Page - MVP version of 'Stay in Control' section (#11245) * Added MVP version of 'Stay in Control' section * PNI CC Page - fixes and improvements (#11242) * fixes and improvements * [PNI CC Page] More fixes (#11248) * optimized face-5.svg * hero creepy face position adjectment --------- Co-authored-by: Daniel Miranda <[email protected]>
This is part of the new PNI index page (Annual Consumer Creep-O-Meter page) work.
Code changes in this PR are going to the feature branch
feature-pni-privacy-index-2023
Related PRs/issues: #11076
Please refer to the checklist in the ticket #11076 . Basically we are adding a new page type for PNI. Content is hardcoded and doesn't need to be editable on CMS. I added a field
year
just because this has the possibility of being an annual thing if this year's launch goes well. This field isn't in the specs so we can def remove it if you see any concern with it.