-
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
Post-merge cleanup #8069
Post-merge cleanup #8069
Conversation
src/components/Layout.tsx
Outdated
@@ -180,7 +165,7 @@ const Layout: React.FC<IProps> = ({ | |||
<Nav path={path} /> | |||
{shouldShowSideNav && <SideNavMobile path={path} />} | |||
</VisuallyHidden> | |||
<BannerNotification | |||
{/* <BannerNotification |
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.
I suspect we should remove the "merge-complete"
string from common.json?
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.
Oops, good catch. Removing string and comment now.
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.
Nice - LGTM other than removing that 1 string!
✅ ethereum-org-website-dev deploy preview ready
|
Description
hero-panda.png
react-rewards
packageRelated Issue
#7075