Skip to content

[QF-5959] Apply QDC branding to Open Graph previews - #25

Closed
AhmedCodeGuy wants to merge 3 commits into
mainfrom
feature/qf-5959-qdc-rebrand-og-production
Closed

AhmedCodeGuy wants to merge 3 commits into
mainfrom
feature/qf-5959-qdc-rebrand-og-production

Conversation

@AhmedCodeGuy

@AhmedCodeGuy AhmedCodeGuy commented Aug 10, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Ticket: QF-5959

Production counterpart of testing PR #24, including testing follow-up #27.

  • Applies the official QDC branding to generated and reviewed premade previews on top of main.
  • Replaces the localized homepage device compositions with one official Quran.com horizontal logo centered on a clean 1200×630 off-white canvas for every locale.
  • Replaces 13 duplicated homepage images with one 13.8 KB shared asset while preserving the official source logo and its aspect ratio.
  • Retains the production-only Sentry-aware internal fetch path for both the default and chapter routes.

Validation

  • yarn playwright test tests/qdc-branding.spec.ts — 4 passed
  • yarn lint — passed
  • yarn build — passed
  • 23 supported and legacy locale values returned HTTP 200, image/png, and the same approved image hash through the production Sentry wrapper
  • Verified 1200×630 output, 780×162 centered logo placement, and safe margins
  • Existing Next.js, Browserslist, and production Sentry edge-runtime build warnings remain unchanged

Release

This PR remains unmerged until the QF-5959 production release date.

Prelive preparation — 4 October 2026

Combined release candidate: #33 integrates this QDC branding with #30, gives the later approved 1200×630 artwork precedence, and preserves current main delivery/edge/server protections. Keep all candidates unmerged until prelive acceptance; main deploys live production.

@AhmedCodeGuy

Copy link
Copy Markdown
Collaborator Author

Superseded by the combined approved artwork candidate #33, which preserves generated-preview QDC branding and current production protections while choosing #30's later approved artwork where the sources overlap. The coverage/selection decision is recorded on #33. Closing this overlapping source without merging; #33 remains open and held for the coordinated production release.

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.

1 participant