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

[feat] 404 Page and dynamic titles #14

Merged
merged 3 commits into from
May 14, 2022
Merged

[feat] 404 Page and dynamic titles #14

merged 3 commits into from
May 14, 2022

Conversation

Hamdrive
Copy link
Owner

@Hamdrive Hamdrive commented May 14, 2022

Overview

Successfully implemented 404 page for routes which are not defined.
Additionally now each page has its own dynamic title generated based on location and auth state

Note to reviewer

Kindly review just this commit 823c525
as others are not relevant/need not be reviewed

@vercel
Copy link

vercel bot commented May 14, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
kemmel-tube ✅ Ready (Inspect) Visit Preview May 14, 2022 at 5:38AM (UTC)

Copy link

@abhij1607 abhij1607 left a comment

Choose a reason for hiding this comment

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

Great work with useDocumentTitle in utils;
Good coding style
Good use of mui components

Copy link

@shivampandey0 shivampandey0 left a comment

Choose a reason for hiding this comment

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

Great work with the custom hook for the document.title

Good to merge

Copy link

@GuptaEesh GuptaEesh left a comment

Choose a reason for hiding this comment

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

  • good work with the custom hook for the windowScrollTop and documentTitle.

Good to merge

@Hamdrive Hamdrive merged commit df4ca44 into dev May 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants