Skip to content

Add profile metadata - #198

Open
piushisharma wants to merge 2 commits into
NST-DEVFORGE:mainfrom
piushisharma:add-profile-metadata
Open

piushisharma wants to merge 2 commits into
NST-DEVFORGE:mainfrom
piushisharma:add-profile-metadata

Conversation

@piushisharma

Copy link
Copy Markdown

What

Added the required profile metadata as requested in issue #70. This updates the profile information and ensures the requested metadata is included in the appropriate place.

Fixes #70

How I tested it

I tested the changes locally after adding the profile metadata. I checked the updated files to make sure the metadata was added correctly and that the existing profile functionality was not affected. I also reviewed the changes to confirm that only the files and sections related to the issue were modified.

Checklist

  • I was assigned the issue before I started
  • npm run lint and npm run build pass locally
  • The diff only touches what the issue asked for

@piushisharma
piushisharma requested a review from a team as a code owner October 7, 2026 08:04
@vercel

vercel Bot commented Oct 7, 2026

Copy link
Copy Markdown

@piushisharma is attempting to deploy a commit to the Geetansh Goyal's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown

📸 How this looks

Built from this PR, so you can see the change without checking it out.

/

Desktop (1280px), top 3000px
Mobile (390px), top 3000px

/learn/open-source

Desktop (1280px), top 3000px
Mobile (390px), top 3000px

Screenshots are taken from a production build of this PR, signed out. Pages behind sign-in show the signed-out view. Build log

github-actions Bot added a commit that referenced this pull request Oct 7, 2026

@bhavesh-210 bhavesh-210 left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM! The implementation follows Next.js 15 conventions with await params, correctly handles the title template using { absolute: ... }, and satisfies all criteria from #70.

Minor note: Don't forget to tick the checkboxes in your PR description checklist (- [x]).

Ready to merge!

This branch has not been deployed

No deployments
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.

Student profile pages: generate a title and description per student

2 participants