betagouv/mon-entreprise

[Artiste-auteur] Dans les pages de documention, le bouton "retourner à la simulation" ne fonctionne pas

Closed this issue · 5 comments

Sur l'ensemble des pages de documentation accessibles en cliquant sur les résultats du simulateur "artiste-auteur", le bouton "retourner à la simulation" ne fonctionne pas. Quand on clique dessus, on a une erreur 404.

Cf. la vidéo, avec comme exemple la page de documentation sur les cotisations Urssaf (le problème est identique pour les autres pages documentation rattachées au simulateur).

Par contre, il est possible de retourner au simulateur avec le bouton "revenir en arrière" du navigateur.

Constat fait avec les navigateurs Firefox et Chrome.

Enregistrement.2024-06-03.082409.mp4

Bonjour,

La valeur utilisée dans useSimulationConfig pour "key" est "artiste-auteur", alors qu'elle devrait correspondre à une URL du simulateur telle que "/simulateurs/artiste-auteur" (d'après l'implémentation sur les autres pages).

Voici ma proposition :

// ArtisteAuteur.tsx
export default function ArtisteAuteur() {
	useSimulationConfig({
		key: '/simulateurs/artiste-auteur', // auparavant "artiste-auteur"
		config: configArtisteAuteur,
	})
}

Si cela vous convient, je fais une pull request. 👍

Avec plaisir @Retroscilo !

Hello @Retroscilo,

Bien vu ! 👍🏻

Je viens de jeter un oeil à ta solution et elle pourrait effectivement faire l'affaire à ceci près que les routes sont pour la plupart internationalisées et que le chemin serait dans ce cas /calculators/artist-author sur le site en anglais.

Tu pourrais par exemple construire le chemin en utilisant le hook useSitePaths, mais à mon avis, récupérer le chemin courant devrait être suffisant ici :

const { pathname } = useLocation();
useSimulationConfig({ key: pathname, config: configArtisteAuteur });

L'url utilisée par le composant BackToSimulation pour rediriger l'utilisateur étant déjà définie comme suit :

const url = useSelector((state: RootState) => state.simulation?.url)

@HichamBenjelloun ,

J'ai raté cette composante en effet ! Merci pour la mise au point, la branche est mise à jour avec ta suggestion 👍

Correction vérifiée en production, merci @Retroscilo ! 🙏🏾