-
Notifications
You must be signed in to change notification settings - Fork 44
Chore/seo improvements 08 03 25 #524
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
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.
PR Overview
This PR refactors SEO metadata for various marketing and app pages by removing the shared WebPageJsonLdBreadcrumb import and replacing it with inline breadcrumb definitions. The changes standardize breadcrumb data across multiple pages to improve SEO consistency.
- Removed WebPageJsonLdBreadcrumb import and usage in favor of inline breadcrumb objects.
- Updated breadcrumb definitions for multiple pages with tailored itemListElement arrays.
- Minor adjustments to breadcrumb item names and URLs for improved clarity.
Reviewed Changes
| File | Description |
|---|---|
| src/app/(marketing)/(landing-pages)/(pillar-pages)/javascript-coding-interview-questions-and-answers/page.tsx | Inline breadcrumb defined replacing shared import. |
| src/app/(marketing)/(landing-pages)/(pillar-pages)/javascript-projects-for-beginners/page.tsx | Inline breadcrumb with updated title. |
| src/app/(marketing)/(landing-pages)/javascript-coding-questions/page.tsx | Inline breadcrumb updated for JavaScript Coding Questions. |
| src/app/(marketing)/(landing-pages)/(pillar-pages)/javascript-cheat-sheet/page.tsx | Inline breadcrumb defined for cheat sheet page. |
| src/app/(marketing)/(landing-pages)/free-leetcode-alternative/page.tsx | Inline breadcrumb updated for Free LeetCode Alternative. |
| src/app/(app)/(default_layout)/(roadmaps)/roadmaps/page.tsx | Updated breadcrumb with inline definition for the Roadmaps page. |
| src/app/(marketing)/(landing-pages)/daily-coding-challenges-for-beginners/page.tsx | Inline breadcrumb updated for Daily Coding Challenges for Beginners. |
| src/app/(marketing)/(landing-pages)/react-coding-challenges/page.tsx | Inline breadcrumb added for React Coding Challenges with metadata generation. |
| src/app/(marketing)/(landing-pages)/learn-javascript/page.tsx | Inline breadcrumb defined for Learn JavaScript. |
| src/app/(marketing)/(landing-pages)/javascript-roadmap/page.tsx | Inline breadcrumb updated for the JavaScript Roadmap page. |
| src/app/(marketing)/(landing-pages)/daily-coding-challenges/page.tsx | Inline breadcrumb updated for Daily Coding Challenges page. |
| src/app/(marketing)/(landing-pages)/coding-roadmap/page.tsx | Inline breadcrumb updated for the Coding Roadmap page. |
| src/app/(marketing)/(landing-pages)/how-to-learn-javascript/page.tsx | Inline breadcrumb defined for How to Learn JavaScript. |
| src/app/(marketing)/(landing-pages)/how-to-learn-to-code/page.tsx | Inline breadcrumb defined for How to Learn to Code. |
| src/app/(marketing)/(landing-pages)/free-coding-challenges/page.tsx | Inline breadcrumb updated for Free Coding Challenges. |
| src/app/(marketing)/(landing-pages)/learn-to-code/page.tsx | Inline breadcrumb defined for Learn to Code page. |
| src/app/(app)/(default_layout)/(questions)/questions/page.tsx | Updated breadcrumb inline for Questions page with an adjusted URL. |
| src/app/(marketing)/(landing-pages)/[...slug]/page.tsx | Inline breadcrumb defined; potential undefined value used in breadcrumb item. |
Copilot reviewed 33 out of 33 changed files in this pull request and generated 1 comment.
| '@type': 'ListItem', | ||
| position: 2, | ||
| name: title, | ||
| item: fullUrl, |
Copilot
AI
Mar 8, 2025
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.
The variable 'fullUrl' is used without being defined. Consider defining 'fullUrl' or replacing it with a properly computed URL.
No description provided.