[FEAT] Add About Page for Company Information (#177)#198
Open
Mrigakshi-Rathore wants to merge 1 commit into
Open
[FEAT] Add About Page for Company Information (#177)#198Mrigakshi-Rathore wants to merge 1 commit into
Mrigakshi-Rathore wants to merge 1 commit into
Conversation
- Create comprehensive About page at /about - Include mission statement and core values - Showcase key features and benefits - Display tech stack information - Add open source contribution section - Match existing LinkID theme with responsive design - Add dark mode support
|
@Mrigakshi-Rathore is attempting to deploy a commit to the vishnukothakapu's projects Team on Vercel. A member of the Team first needs to authorize it. |
Contributor
📝 WalkthroughWalkthroughA new About page ( ChangesAbout Page Feature
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
🧹 Nitpick comments (1)
app/about/page.tsx (1)
63-99: ⚡ Quick winConsolidate repeated card markup into mapped data structures.
The mission and feature cards duplicate the same container/icon/text structure. Moving content into typed arrays and rendering with
mapwill reduce drift and simplify future edits.♻️ Proposed refactor sketch
+const missionPillars = [ + { title: "Simplify", desc: "Replace long, messy URLs with clean, professional links", icon: Link2 }, + { title: "Connect", desc: "Bring all your professional platforms under one username", icon: Users }, + { title: "Grow", desc: "Build your professional brand with a memorable presence", icon: Rocket }, +]; + +const features = [ + { title: "Platform Routing", desc: "Predictable links like /github, /linkedin, /leetcode for every professional platform you use.", icon: Route }, + { title: "One-Time Setup", desc: "Add links once. Share everywhere. Update anytime without breaking existing links.", icon: Zap }, + { title: "Secure & Private", desc: "OAuth authentication with Google & GitHub. Your data stays secure and private.", icon: Shield }, + { title: "Multi-Platform Support", desc: "GitHub, LinkedIn, LeetCode, YouTube, Twitter, and 10+ other platforms supported.", icon: Globe }, +];-<div className="grid md:grid-cols-3 gap-8"> - ...three repeated mission cards... -</div> +<div className="grid gap-8 md:grid-cols-3"> + {missionPillars.map(({ title, desc, icon: Icon }) => ( + <div key={title} className="text-center p-6 rounded-2xl border border-white/70 bg-white/70 backdrop-blur-xl dark:border-white/10 dark:bg-white/[0.04]"> + <div className="flex justify-center mb-4"> + <div className="p-3 rounded-xl bg-gradient-to-br from-violet-100 to-indigo-100 text-violet-700 dark:from-violet-400/15 dark:to-indigo-400/10 dark:text-violet-200"> + <Icon className="h-6 w-6" /> + </div> + </div> + <h3 className="text-lg font-semibold text-zinc-950 dark:text-white mb-2">{title}</h3> + <p className="text-sm text-zinc-600 dark:text-zinc-400">{desc}</p> + </div> + ))} +</div>Also applies to: 115-170
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@app/about/page.tsx` around lines 63 - 99, The three repeated feature cards in about/page.tsx (the grid div rendering three identical card blocks with icons Link2, Users, Rocket and headings "Simplify", "Connect", "Grow") should be replaced by a typed array of card objects (e.g., { key, title, description, Icon }) and a single map over that array to render the shared card markup; ensure you pass the Icon component reference (Link2, Users, Rocket) from the data, include a stable key prop, and preserve all existing className styling and accessibility attributes. Apply the same pattern to the other duplicate blocks mentioned (lines ~115-170) so all mission/feature card groups are driven from data arrays and rendered with a single reusable mapping to reduce duplication and drift.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@app/about/page.tsx`:
- Around line 63-99: The three repeated feature cards in about/page.tsx (the
grid div rendering three identical card blocks with icons Link2, Users, Rocket
and headings "Simplify", "Connect", "Grow") should be replaced by a typed array
of card objects (e.g., { key, title, description, Icon }) and a single map over
that array to render the shared card markup; ensure you pass the Icon component
reference (Link2, Users, Rocket) from the data, include a stable key prop, and
preserve all existing className styling and accessibility attributes. Apply the
same pattern to the other duplicate blocks mentioned (lines ~115-170) so all
mission/feature card groups are driven from data arrays and rendered with a
single reusable mapping to reduce duplication and drift.
Contributor
Author
|
Hi @vishnukothakapu, I have put significant effort into this PR including:
Request you to kindly assign the appropriate level (medium/advanced) based on the effort put into this contribution. Thank you for your consideration! 🙏 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Anushreebasics
requested changes
May 23, 2026
Anushreebasics
left a comment
Anushreebasics
left a comment
Collaborator
There was a problem hiding this comment.
PR description mentions "Terms of Service" in one place — update to "About page" for clarity.
Anushreebasics
requested changes
May 23, 2026
| Rocket, | ||
| Heart, | ||
| Star, | ||
| Code2, |
Collaborator
There was a problem hiding this comment.
remove unsed imports
|
|
||
| <div className="mx-auto max-w-4xl text-center"> | ||
| <div className="inline-flex items-center gap-2 rounded-full border border-violet-200/70 bg-white/70 px-4 py-2 text-sm font-medium text-violet-700 shadow-sm backdrop-blur-xl dark:border-violet-400/20 dark:bg-white/5 dark:text-violet-200 mb-6"> | ||
| <Rocket className="h-4 w-4" /> |
Collaborator
There was a problem hiding this comment.
mark decorative icons as hidden from assistive tech (e.g. ) or provide accessible labels for interactive icons; ensure interactive controls have visible focus styles (add focus-visible:ring classes).
Anushreebasics
requested changes
May 23, 2026
Anushreebasics
left a comment
Anushreebasics
left a comment
Collaborator
There was a problem hiding this comment.
consider adding openGraph metadata or twitter: metadata if desired; copy looks fine but could be tightened for SEO/snippets.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Adds a comprehensive About page at
/aboutexplaining LinkID's purpose, vision, and core features.Changes Made
app/about/page.tsxwith complete company informationSections Included
Related Issue
Closes #177
Screenshots
Checklist