Skip to content
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

M24 remove ctas (fix #15478) #15493

Merged
merged 1 commit into from
Nov 13, 2024
Merged

Conversation

wen-2018
Copy link
Collaborator

One-line summary

This PR removes two CTAs from M24 home and about pages.

Significant changes and points to review

http://localhost:8000/en-US/
http://localhost:8000/en-US/about/

Issue / Bugzilla link

#15478

Testing

Turn on switch M24_WEBSITE_REFRESH

@wen-2018 wen-2018 assigned wen-2018 and unassigned wen-2018 Nov 13, 2024
@wen-2018 wen-2018 added Needs Review Awaiting code review Review: µ Code review time: 5 minutes or less Frontend HTML, CSS, JS... client side stuff Refresh 🦖 Work related to the 2024 site refresh labels Nov 13, 2024
@maureenlholland maureenlholland self-assigned this Nov 13, 2024
Copy link
Collaborator

@maureenlholland maureenlholland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 r+

@maureenlholland maureenlholland merged commit a3812e8 into mozilla:main Nov 13, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend HTML, CSS, JS... client side stuff Needs Review Awaiting code review Refresh 🦖 Work related to the 2024 site refresh Review: µ Code review time: 5 minutes or less
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants