fix(auth): redirect authenticated users to "Parc de logements" when accessing guest-only routes #1036
Conversation
| @@ -12,7 +13,7 @@ function RequireGuest(props: PropsWithChildren<RequireGuestProps>) { | |||
| return props.children; | |||
| } | |||
|
|
|||
| return <NotFoundView />; | |||
| return <Navigate to="/parc-de-logements" replace />; | |||
There was a problem hiding this comment.
/parc-de-logements est inaccessible quand on n’est pas authentifié. Il faudrait plutôt rediriger vers /connexion
There was a problem hiding this comment.
De plus, le bug dans la carte indiquerait plutôt un mauvais lien dans le site vitrine. Son bouton Connexion devrait amener directement sur /connexion
There was a problem hiding this comment.
Le lien est correct sur le site vitrine. L'erreur se produit quand l'utilisateur est déjà connecté: la route /connexion est inaccessible. Ce correctif permet de rediriger l'utilisateur connecté vers /parc-de-logements, initiative que j'ai prise.
There was a problem hiding this comment.
Le code fonctionne ainsi : si l'utilisateur n'est pas connecté (isGuest) alors on le redirige vers la vue prévue. En revanche pour toute URL "invité", s'il est connecté on le dirige vers le parc logement.
En résumé, historiquement on affichait NotFound pour toute tentative de l'utilisateur connecté d'accéder à une route invité. Avec ce correctif, dorénavant on le redirige vers le parc logement. Le comportement pour les utilisateurs non connectés n'est pas modifié.
…ccessing guest-only routes
Falinor
force-pushed
the
fix-login-when-logged
branch
from
e731b96 to
22c4889
Compare
No description provided.