-
Notifications
You must be signed in to change notification settings - Fork 153
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
added web monetization message to foundation site #7713
Conversation
This PR introduces visual differences. Click here to inspect the diffs. |
Agreed with pomax's suggestion but the tailwind classes look correct 👍🏾 |
Co-authored-by: Pomax <pomax@nihongoresources.com>
This PR introduces visual differences. Click here to inspect the diffs. |
1 similar comment
This PR introduces visual differences. Click here to inspect the diffs. |
I can approve this for design since Nancy is out today. The design looks great and I will approve it but I just double checked with Anil and the footer link should actually also be on https://mozfest-foundation-s-7141-web-m-yafyg6.mofostaging.net/ and https://foundation-s-7141-web-m-yafyg6.herokuapp.com/privacynotincluded but it can be done as a follow up if that's easier. |
Thanks @kristinashu! @Pomax does that mean I should remove the flag for displaying the monetization text and just display it on pni/mozfest/mofo-site? |
This PR introduces visual differences. Click here to inspect the diffs. |
2 similar comments
This PR introduces visual differences. Click here to inspect the diffs. |
This PR introduces visual differences. Click here to inspect the diffs. |
Per request of Kristina, we now made the new monetization message display on all pages of the foundation site including Mozfest and PNI. |
Closes #7141
Related PRs/issues #
Link to sample test page: https://foundation-s-7141-web-m-yafyg6.herokuapp.com/en/
Steps to test: