Skip to content

fixed opengraph bugs#29

Merged
T4RuN05 merged 1 commit into
mainfrom
preprod
May 28, 2026
Merged

fixed opengraph bugs#29
T4RuN05 merged 1 commit into
mainfrom
preprod

Conversation

@T4RuN05

@T4RuN05 T4RuN05 commented May 28, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings May 28, 2026 18:33
@vercel

vercel Bot commented May 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
lgi Ready Ready Preview, Comment May 28, 2026 6:33pm

@T4RuN05
T4RuN05 merged commit cc3ecde into main May 28, 2026
2 checks passed

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 and Twitter card metadata for shared cart pages and sets a global metadataBase URL for proper resolution of relative image URLs in metadata.

Changes:

  • Set metadataBase in root layout metadata to enable absolute URL resolution.
  • Add Open Graph and Twitter image metadata to shared cart pages pointing to the dynamic opengraph-image route.
  • Update shared cart titles by removing the "| The Curated Decor" suffix.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/app/layout.js Adds metadataBase for proper absolute URL resolution in metadata.
src/app/cart/share/[id]/page.jsx Adds OG/Twitter image metadata and simplifies titles.

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

Comment thread src/app/layout.js
Comment on lines +23 to 24
metadataBase: new URL("https://www.lordganeshaimpex.com"),
title: "Lord Ganesha Impex | Premium Jewelry Manufacturer & Global Supplier",
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.

2 participants