chore(web): document set-state-in-effect on ThemeToggle localStorage read (#90)
Review — APPROVE
Verdict : APPROVE
Changement comment-only validé. Le directive eslint-disable-next-line react-hooks/set-state-in-effect cible la bonne ligne et supprime une erreur…
fix(security): override @expo/cli ws to ^8.20.1 (GHSA-58qx-3vcg-4xpx) (#92)
chore(web): document set-state-in-effect on ThemeToggle localStorage read (#90)
maximus
created branch issue-90-themetoggle-lint-disable in maximus/simpl-liste
2026-05-30 18:57:46 +00:00
maximus
pushed to issue-90-themetoggle-lint-disable at maximus/simpl-liste
2026-05-30 18:57:46 +00:00
chore(web): fix prefer-const and remove unused import
fix(web): resolve display name from userInfo, not just claims (#70)
Harmoniser le display name avec la vitrine (userInfo au lieu de claims)
chore(web): fix prefer-const and remove unused import
Review — APPROVE
Trois corrections lint mécaniques et pré-existantes dans web/. Toutes les affirmations de la PR vérifiées contre le working tree et le tooling.
Vérification
-…
[security] deps: vulnerability in ws (medium)
chore(web): fix prefer-const and remove unused import
ThemeToggle: lint react-hooks/set-state-in-effect (hydratation localStorage)
fix(web): resolve display name from userInfo, not just claims (#70)
/pr-review — APPROVE
Summary — Correct, minimal, type-safe fix. The display-name resolution is moved to the userInfo endpoint with the ID-token claims preserved as fallback; the change…