Skip to content

Add Cookie consent, migration prefix collision, seo & open graph metadata audit#658

Open
DioChuks wants to merge 3 commits into
Suncrest-Labs:mainfrom
DioChuks:chore/doc-seo-migration
Open

Add Cookie consent, migration prefix collision, seo & open graph metadata audit#658
DioChuks wants to merge 3 commits into
Suncrest-Labs:mainfrom
DioChuks:chore/doc-seo-migration

Conversation

@DioChuks
Copy link
Copy Markdown
Contributor

@DioChuks DioChuks commented Jun 4, 2026

Summary

  1. Global OG and Twitter Cards

    • Audited the root layout.tsx and cleaned up hardcoded canonical URLs and Open Graph URLs so Next.js can dynamically resolve them per-page using metadataBase.
    • The global layout properly configures the base openGraph and twitter:card (summary_large_image) for all routes.
  2. Canonical URLs (<link rel="canonical">)

    • Homepage: Added an explicit metadata export to the homepage with alternates: { canonical: "/" }.
    • Docs Pages: Updated the generateMetadata function in the docs route to ensure every dynamically generated documentation page points to its correct canonical path (/docs/[slug]), and added a fallback for the /docs landing page.
  3. OG Image (1200x630)

    • Generated a highly professional, 1200x630 Open Graph image featuring decentralized finance themes, dark mode aesthetics, and the Nester brand.
    • I replaced the existing image at apps/website/public/og-image.png with this new graphic.
  4. JSON-LD Structured Data

    • Audited the homepage and verified it injects rich JSON-LD structured data for both a WebApplication (with offers and FinanceApplication categories) and an FAQPage schema. The global layout also continues to inject the core Organization schema.
  5. Build Verification

    • Successfully ran pnpm run build locally. The static and dynamic route prerendering finished perfectly, confirming there are no Next.js metadata API errors. These upgrades guarantee that all pages satisfy Google's Rich Results standards and should push your Lighthouse SEO score >90.

I've successfully added the migration prefix collision check and resolved the remaining collisions in apps/api/migrations/.
the static localStorage banner was replaced with a dynamic, GDPR-compliant consent manager. It uses document.cookie, natively integrates a Preferences link into your footer, and conditionally injects the Plausible analytics tracker depending on the user's opt-in choices.

Related Issues

Closes #634
Closes #644
Closes #635

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactor
  • Documentation

Changes Made

Testing Done

Ran build & successful

Screenshots

For UI changes, add before/after screenshots if applicable.

Checklist

  • Code follows the project's style guidelines
  • Self-review completed
  • Tests added/updated for changes
  • Documentation updated if needed
  • No secrets or credentials in the code
  • Smart contract changes reviewed for security implications

@DioChuks DioChuks requested a review from 0xDeon as a code owner June 4, 2026 19:04
@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Jun 4, 2026

@DioChuks Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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

Labels

None yet

Projects

None yet

1 participant