Skip to content

feat(a11y): plan du site RGAA 12.1#204

Open
skelz0r wants to merge 1 commit into
developfrom
site/a11y-sitemap
Open

feat(a11y): plan du site RGAA 12.1#204
skelz0r wants to merge 1 commit into
developfrom
site/a11y-sitemap

Conversation

@skelz0r

@skelz0r skelz0r commented Jun 18, 2026

Copy link
Copy Markdown
Member

RGAA 12.1 — plan du site fonctionnel (le lien footer pointait vers `href="#"`).

`config/sitemap.yml` source de vérité unique (sections + chemins + titres), rendu par une vue partagée, protégé par un spec qui valide chaque chemin contre le routeur Rails — impossible d'ajouter une page sans route sans casser la CI.

Closes API-6732

sitemap-screenshot

@linear

linear Bot commented Jun 18, 2026

Copy link
Copy Markdown

API-6732

@skelz0r skelz0r self-assigned this Jun 18, 2026
@skelz0r skelz0r force-pushed the develop branch 2 times, most recently from d11e169 to 538b030 Compare June 19, 2026 10:20
@skelz0r skelz0r force-pushed the site/a11y-sitemap branch from ba688dd to a3ddc68 Compare June 22, 2026 07:55
RGAA 12.1 requires two navigation systems; we had a header nav but no
sitemap (the footer link was a dead `href="#"`).

This adds:
- `config/sitemap.yml` — authoritative list of public pages for both API
  Entreprise and API Particulier, grouped by section. Each entry references
  a route by its helper name, not a hardcoded path, so links follow
  automatically when a route's path changes (same convention as changelogs).
- `/plan-du-site` route + `pages#sitemap` action in both namespaces.
- `shared/pages/sitemap.html.erb` — shared view resolving each entry via its
  route helper.
- Footer links updated from `href="#"` to the real path helper.
- `spec/features/sitemap_spec.rb` — visits both /plan-du-site pages; since
  rendering resolves every route helper, a removed or renamed route blows up
  the page and fails the spec, with no separate route check to maintain.

Closes https://linear.app/datagouv/issue/API-6732
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