Code-review fixes and refactors #26

Merged
eliott merged 4 commits from fix/code-review into main 2026-10-15 22:39:46 +02:00
Owner

Code-review pass for the marketing site. Release 0.6.0.

Includes the brand update already on main/develop.

  • /changelog is server-rendered with caching and a clear "unavailable" state (no longer frozen at build time)
  • Styled error page, nav active state + aria-current, single , mobile menu as a modal dialog, inert FAQ answers
  • Docs layout loads its nav server-side (about 35% less JS on docs pages)
  • Data-driven features table, LegalPage, shared CTA and site config, legal page AGPL wording
  • Self-hosting docs: Postgres 18 volume path and config drift fixed
  • SEO: descriptions, breadcrumbs, distinct screenshots with srcset, font preloads
  • Changelog types generated from rollnote-api's openapi.json
  • Infra: Docker image builds without .env (PUBLIC_* build args), runs as node, no node_modules; CI actions pinned, gitleaks, docker build
Code-review pass for the marketing site. Release **0.6.0**. Includes the brand update already on main/develop. - /changelog is server-rendered with caching and a clear "unavailable" state (no longer frozen at build time) - Styled error page, nav active state + aria-current, single <main>, mobile menu as a modal dialog, inert FAQ answers - Docs layout loads its nav server-side (about 35% less JS on docs pages) - Data-driven features table, LegalPage, shared CTA and site config, legal page AGPL wording - Self-hosting docs: Postgres 18 volume path and config drift fixed - SEO: descriptions, breadcrumbs, distinct screenshots with srcset, font preloads - Changelog types generated from rollnote-api's openapi.json - Infra: Docker image builds without .env (PUBLIC_* build args), runs as node, no node_modules; CI actions pinned, gitleaks, docker build
- Dockerfile takes the PUBLIC_* values as build args (the image could not
  build without .env), runs as node, ships only build/ and package.json,
  healthcheck, pinned base images; hardened compose service
- CI: actions pinned by SHA, node from .nvmrc, gitleaks, docker build,
  generated API types drift check
- security headers are set by the reverse proxy (documented)
- remove the committed .idea/ project files
feat: code-review fixes - SSR changelog, error page, a11y, smaller docs bundle (0.6.0)
All checks were successful
tests / test (pull_request) Successful in 1m38s
11ed2e7a1b
- /changelog renders per request with caching and a clear "unavailable"
  state instead of a build-time snapshot
- styled 404/error page, nav active state and aria-current fixed, one
  <main> per page, mobile menu as a modal dialog, inert FAQ answers
- docs layout loads its nav server-side (about 35% less JS per docs page)
- data-driven features table, LegalPage, shared CTA and site config
- self-hosting docs: Postgres 18 volume path and config drift fixed
- SEO: descriptions, breadcrumbs, distinct screenshots with 1x/2x srcset,
  font preloads
- changelog types generated from rollnote-api's openapi.json
- release 0.6.0
eliott merged commit 3f4ee6ebc2 into main 2026-10-15 22:39:46 +02:00
eliott deleted branch fix/code-review 2026-10-15 22:39:46 +02:00
Sign in to join this conversation.
No reviewers
No labels
run-ci
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
RollNote/rollnote-website!26
No description provided.