Skip to content

Feat: Inject dynamic SEO meta tags and JSON-LD schema - #598

Closed
DebasmitaBose0 wants to merge 1 commit into
komalharshita:mainfrom
DebasmitaBose0:enhancement/seo-meta-project
Closed

Feat: Inject dynamic SEO meta tags and JSON-LD schema#598
DebasmitaBose0 wants to merge 1 commit into
komalharshita:mainfrom
DebasmitaBose0:enhancement/seo-meta-project

Conversation

@DebasmitaBose0

Copy link
Copy Markdown

Closed #597

Problem:
The project.html template does not feature dynamic Open Graph (OG) meta tags or JSON-LD schema markup. When a specific project is shared on social media or crawled by search engines, it falls back to generic site metadata.
Acceptance Criteria:

  • Inject Open Graph (og:title, og:description) <meta> tags into the document <head>.
  • Dynamically populate these tags using Jinja2 variables specific to the requested project.

Copilot AI review requested due to automatic review settings May 25, 2026 16:22
@vercel

vercel Bot commented May 25, 2026

Copy link
Copy Markdown

@DebasmitaBose0 is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copilot AI 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.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds Open Graph metadata to the project detail page to improve link previews when the page is shared.

Changes:

  • Add og:title meta tag using project.title
  • Add og:description meta tag using project.description

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread templates/project.html
Comment on lines +24 to +25
<meta property="og:title" content="{{ project.title }}" />
<meta property="og:description" content="{{ project.description }}" />
Comment thread templates/project.html
Comment on lines +24 to +25
<meta property="og:title" content="{{ project.title }}" />
<meta property="og:description" content="{{ project.description }}" />

@komalharshita komalharshita left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Good direction overall — adding dynamic SEO metadata to project pages is a meaningful improvement for social sharing and discoverability.

A few important issues should be addressed before merge:

  • The PR title mentions JSON-LD schema support, but no actual JSON-LD structured data was implemented yet.

  • Current Open Graph support is incomplete. Right now only:

    • og:title
    • og:description

    were added.

    Important tags like:

    • og:type
    • og:url
    • og:image
    • twitter:card
    • Twitter metadata

    are still missing.

  • project.description is injected directly into HTML attributes and should ideally be sanitized/truncated for safer preview rendering.

  • Fallback values should exist if project title/description are missing.

  • Please include validation/testing evidence (social preview screenshots or validator results) for SEO-related changes.

The feature idea is good, but the implementation needs to be completed/refined before approval.

@komalharshita

Copy link
Copy Markdown
Owner

@DebasmitaBose0 use of Copilot AI or any other AI agents/tools is not allowed while opening PRs for GSSoC.

This PR contains visible Copilot-generated review suggestions and AI-assisted code patterns in the commit/review flow, which goes against the contribution guidelines for this repository/program.

Please make sure all future contributions are:

  • independently written and understood by you
  • manually reviewed and tested
  • free from AI-generated/copilot-assisted implementation or review dependency

Repeated violations may lead to PR disqualification from the program. Please follow the GSSoC contribution rules strictly going forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dynamic SEO Meta Tags & Breadcrumbs Schema

3 participants