Skip to content

fix: resolve metadata URL and remove raw HTML - #85

Merged
joelpeace48-cell merged 2 commits into
FinesseStudioLab:mainfrom
telemarkdigital-publisher:fix/issues-1-2-metadata-raw-html
Aug 25, 2026
Merged

fix: resolve metadata URL and remove raw HTML#85
joelpeace48-cell merged 2 commits into
FinesseStudioLab:mainfrom
telemarkdigital-publisher:fix/issues-1-2-metadata-raw-html

Conversation

@telemarkdigital-publisher

Copy link
Copy Markdown
Contributor

Summary

  • Resolves metadataBase from NEXT_PUBLIC_SITE_URL or Vercel's deployment URL instead of hardcoding localhost.
  • Adds root canonical and Open Graph URL metadata.
  • Replaces the landing-page dangerouslySetInnerHTML with equivalent JSX.
  • Enables react/no-danger and adds unit coverage for metadata URL resolution and the raw-HTML guard.

Closes #1
Closes #2

Verification

  • npm test
  • npm run lint
  • npm run build
  • git diff --check

Notes

@joelpeace48-cell

Copy link
Copy Markdown
Contributor

@telemarkdigital-publisher ci fails, please fix it

@telemarkdigital-publisher

Copy link
Copy Markdown
Contributor Author

Thanks for the heads-up. I pushed a narrow CI/accessibility fix in 716ffb5.

The failing axe rule was scrollable-region-focusable on the wide chart plot in /operators under the mobile Safari project. The chart frame now gives wide scrollable plots keyboard focus and a visible focus outline, so the horizontal scroll region is reachable without a pointer.

Verification run locally:

  • npm test
  • npm run lint
  • npm run build
  • npx playwright test e2e/accessibility.spec.ts --project=chromium --grep operators

I could not run the local mobile-safari project on this Windows workspace because the local Playwright WebKit browser is not installed here, but the same axe failure path should rerun in CI on the refreshed branch.

@joelpeace48-cell
joelpeace48-cell merged commit f8160d5 into FinesseStudioLab:main Aug 25, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants