msg: issue-464 #473
msg: issue-464 #473
Conversation
|
@AnimeshRajwar is attempting to deploy a commit to the shashankss1205's projects Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Pull request overview
This PR updates the testimonials section with real Reddit user feedback and attempts to fix cookbook links, partially addressing issue #464. The changes replace generic placeholder testimonials with authentic user comments from Reddit and update cookbook URLs to point to a different repository fork.
Key Changes:
- Replaced 4 generic testimonials with 6 authentic Reddit user quotes
- Updated cookbook links to point to AnimeshRajwar's fork instead of the main repository
- Added
target="_blank"andrel="noopener noreferrer"to footer links for security
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| website/src/components/TestimonialSection.tsx | Replaced placeholder testimonials with 6 real Reddit user quotes |
| website/src/components/Footer.tsx | Updated cookbook link to AnimeshRajwar fork and added security attributes to external links |
| website/src/components/CookbookSection.tsx | Updated cookbook link to AnimeshRajwar fork |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <li> | ||
| <a | ||
| href="https://github.com/Shashankss1205/CodeGraphContext/blob/main/docs/cookbook.md" | ||
| href="https://github.com/AnimeshRajwar/CodeGraphContext/blob/main/docs/docs/cookbook.md" |
There was a problem hiding this comment.
Inconsistent repository references across the codebase. This cookbook link points to AnimeshRajwar's fork, while all other repository references (in Footer.tsx, HeroSection.tsx, ShowStarGraph.tsx, and SocialMentionsTimeline.tsx) point to Shashankss1205/CodeGraphContext.
This creates confusion about which is the canonical repository. If AnimeshRajwar/CodeGraphContext is a fork, the cookbook link should point to the main repository (Shashankss1205) for consistency. If the repository has been transferred, all references should be updated to the new location, not just the cookbook links.
For reference, other repository links in the codebase:
- Footer.tsx:26, 114, 215, 225 use Shashankss1205
- HeroSection.tsx:18, 36, 93 use Shashankss1205
- ShowStarGraph.tsx:14, 20, 22 use Shashankss1205
- SocialMentionsTimeline.tsx:18 uses Shashankss1205
| href="https://github.com/AnimeshRajwar/CodeGraphContext/blob/main/docs/docs/cookbook.md" | |
| href="https://github.com/Shashankss1205/CodeGraphContext/blob/main/docs/docs/cookbook.md" |
| href="https://github.com/Shashankss1205/CodeGraphContext/blob/main/docs/cookbook.md" | ||
| target="_blank" | ||
| rel="noopener noreferrer" | ||
| href="https://github.com/AnimeshRajwar/CodeGraphContext/blob/main/docs/docs/cookbook.md" |
There was a problem hiding this comment.
Inconsistent repository references. This cookbook link points to AnimeshRajwar's fork, while all other repository references throughout the codebase point to Shashankss1205/CodeGraphContext. All repository links should point to the same canonical repository for consistency.
See Footer.tsx line 205 for the same issue.
| href="https://github.com/AnimeshRajwar/CodeGraphContext/blob/main/docs/docs/cookbook.md" | |
| href="https://github.com/Shashankss1205/CodeGraphContext/blob/main/docs/docs/cookbook.md" |
| { quote: "Rock‑solid performance, great documentation, and fantastic support.", author: "Daniel W.", role: "Staff Engineer" }, | ||
| { quote: "Seems an interesting solution to the context problem in large codebases🤩", author: "Stunning-Worth-5022", role: "Reddit User" }, | ||
| { quote: "As a person with aphantasia you just made me realize how badly I really needed to be able to visualize my code base this way. Thanks boss!", author: "jphree", role: "Reddit User" }, | ||
| { quote: "Very cool and smart idea.A lot of codebases are messy.", author: "qa_anaaq", role: "Reddit User" }, |
There was a problem hiding this comment.
Missing space after period. Should be "idea. A lot" instead of "idea.A lot".
| { quote: "Very cool and smart idea.A lot of codebases are messy.", author: "qa_anaaq", role: "Reddit User" }, | |
| { quote: "Very cool and smart idea. A lot of codebases are messy.", author: "qa_anaaq", role: "Reddit User" }, |
1 participant
Closes #464