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: optimizing App router implementation for better SEO #1916

Merged
merged 2 commits into from
Jul 31, 2023

Conversation

s2sharpit
Copy link
Contributor

Related Issue

Closes: #1906

Description

Optimizing app router for better SEO

Screenshots

Signed-off-by: Tushar Saini <[email protected]>
@vercel
Copy link

vercel bot commented Jul 31, 2023

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

Name Status Preview Updated (UTC)
projectshut ✅ Ready (Inspect) Visit Preview Jul 31, 2023 7:18pm

@github-actions github-actions bot added the wait for reviewers wait for the maintainer label Jul 31, 2023
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Great job, @s2sharpit! 🎉 Thank you for submitting your first pull request. Your contribution is valuable and we appreciate your efforts to improve our project.

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

@priyankarpal priyankarpal added gssoc23 only for Girlscript summer of code level3 high level fix for GSSoC labels Jul 31, 2023
@priyankarpal
Copy link
Owner

@s2sharpit I'll check it on my local system shortly, and then I'll provide you with an update.

Copy link
Owner

@priyankarpal priyankarpal left a comment

Choose a reason for hiding this comment

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

we can move ProjectList.tsx to components/projects/

@priyankarpal priyankarpal added the changes-required changes-required label Jul 31, 2023
@s2sharpit
Copy link
Contributor Author

we can move ProjectList.tsx to components/projects/

Sure, I have done that. Kindly check it.

Copy link
Owner

@priyankarpal priyankarpal left a comment

Choose a reason for hiding this comment

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

superb! 👏🏻 🔥

@priyankarpal priyankarpal removed the changes-required changes-required label Jul 31, 2023
@priyankarpal priyankarpal merged commit 20ac804 into priyankarpal:main Jul 31, 2023
@s2sharpit s2sharpit deleted the fix-issue-1906 branch July 31, 2023 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc23 only for Girlscript summer of code level3 high level fix for GSSoC wait for reviewers wait for the maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: Enhancing Application Performance with Next.js Server Components
2 participants