Skip to content

feat: simple app router with protected admin route#30

Merged
jbuiquan merged 1 commit into
mainfrom
feat/app-route-provider
Feb 6, 2026
Merged

feat: simple app router with protected admin route#30
jbuiquan merged 1 commit into
mainfrom
feat/app-route-provider

Conversation

@jbuiquan
Copy link
Copy Markdown
Collaborator

@jbuiquan jbuiquan commented Feb 5, 2026

Issue #18

  • Ajout de la lib de routing react-router-dom
  • Passage des routes Admin et Login en lazy loading. Il a fallu les faire passer en export par défaut.
  • AdminRoute est un garde route qui vérifie pour le moment simplement si l'user est authentifié. Il redirige vers la page de login si ce n'est pas le cas.
  • Le bouton de connexion permet de naviguer sur la page de login.

Remarques :

  • J'aurai voulu utiliser <Link/> plutôt qu'un bouton car un lien est sémantiquement plus correct mais j'ai eu des soucis avec les styles tailwind pour qu'il garde la même tête que les boutons.
  • Je n'ai pas pris le temps de créer un Layout pour hériter de la même structure entre page mais j'imagine que ça sera à venir.

@jbuiquan jbuiquan requested a review from arnaudfnr February 5, 2026 21:43
@jbuiquan jbuiquan self-assigned this Feb 5, 2026
@jbuiquan jbuiquan added the enhancement New feature or request label Feb 5, 2026
@arnaudfnr
Copy link
Copy Markdown
Collaborator

J'ai approuvé la PR car elle me convient en l'état. Par curiosité, as-tu essayé avec plutôt que ? https://reactrouter.com/api/components/NavLink

@jbuiquan
Copy link
Copy Markdown
Collaborator Author

jbuiquan commented Feb 6, 2026

J'ai approuvé la PR car elle me convient en l'état. Par curiosité, as-tu essayé avec plutôt que ? https://reactrouter.com/api/components/NavLink

Comme discuté j'ai des problèmes de style avec les liens, les classes tailwind ne s'appliquaient pas sur le lien qui gardait le style par défaut défini dans webapp/src/index.css. Par exemple on a :

a {
    font-weight: 500;
    color: var(--primary);
    text-decoration: inherit;
}

Et je n'arrivais pas a surcharger la couleur avec une classe tailwind. Ce qui fait que sur le bouton "Se connecter" qui est défini avec un bg-primary, le texte avait la même couleur que le fond (entre autres).

@jbuiquan jbuiquan force-pushed the feat/app-route-provider branch from c3e0a7b to 5324815 Compare February 6, 2026 21:07
@jbuiquan jbuiquan merged commit a39ae8c into main Feb 6, 2026
1 check passed
@jbuiquan jbuiquan deleted the feat/app-route-provider branch February 6, 2026 21:10
@arnaudfnr arnaudfnr linked an issue Feb 7, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[D4T Frontend] Feature - Add RoutesProvider

2 participants