-
Notifications
You must be signed in to change notification settings - Fork 193
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
MPP-3616 Initial removal of banners we no longer need #4179
Conversation
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.
Correct. All that is blocking me here is the test coverage threshold - its at like 49% out of 50% required. So its failing. I have to write tests to increase that coverage. |
This PR fixes MPP-3616
New feature description
This PR removes many banners.
Screenshot (if applicable)
Following banners removed:
How to test
Visit profile and make sure these banners do not show up anymore.
Checklist (Definition of Done)
/frontend/src/styles/tokens.scss
).