-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Added banner on community page #11319
Conversation
✅ ethereum-org-website-dev deploy preview ready
|
@rohan9024 I wanted to review this, but I see there are a lot of commits from #11075 in this PR. Looking at your nav PR above, I see that was from your dev branch, so when you made this branch it included all those commits. If you can, those commits should be removed from this PR. If you don't have time today that is totally understandable, I will make a new PR with this if so for this release. |
I wanted to ask whether the changes I have done in this PR is right? Then I can merge it with my dev branch. |
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.
Thanks @rohan9024
I just updated this to also show on the homepage.
Fixes #11316
Referenced this previous PR- #10078
The banner is displayed on community page
Screenshots-