-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
Create Footer Component #5983
Create Footer Component #5983
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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 worked on this section a little bit yesterday and added my comments as I saw it. Great work btw!
FYI @fa-901 linting is failing... (Maybe you need to cast the imported JSON within the siteProvider |
added casting in |
Description
Created a Footer component which resolves issue 5980.
Included translations for links where needed
Made edits to previously created NavItem component to support children & classnames
Validation
Added styles and URLs mentioned in aforementioned issue. Added support for both large & mobile screens. Some screenshots below:
Related Issues
Addresses 5980
Check List
npx turbo lint
to ensure the code follows the style guide. And runnpx turbo lint:fix
to fix the style errors if necessary.npx turbo format
to ensure the code follows the style guide.npx turbo test
to check if all tests are passing.