Skip to content

feat(auth): « remember me » au login - #73

Merged
RebelliousSmile merged 1 commit into
mainfrom
claude/remember-me-login
Jul 31, 2026
Merged

feat(auth): « remember me » au login#73
RebelliousSmile merged 1 commit into
mainfrom
claude/remember-me-login

Conversation

@RebelliousSmile

Copy link
Copy Markdown
Owner

Résumé

Le login persistait toujours le token → l'app te gardait connecté sans choix. Ajoute le choix « remember me ».

  • Case à cocher sur le formulaire de login (cochée par défaut = comportement actuel).
  • Cochée → token en localStorage (survit au redémarrage) + username mémorisé et pré-rempli au prochain login.
  • Décochée → token en sessionStorage (effacé à la fermeture de l'app), username non mémorisé.
  • checkAuth restaure depuis l'un ou l'autre stockage ; logout nettoie les deux.

Tests

  • remember=false → token en sessionStorage, pas en localStorage, username non mémorisé.
  • défaut → username mémorisé + exposé via rememberedUsername.
  • checkAuth restaure un token session-only.
  • 263 tests front · vue-tsc OK · lint : 0 warning ajouté.

🤖 Generated with Claude Code


Generated by Claude Code

Login always persisted the token, so the app remembered you unconditionally.
Add the choice:
- Checkbox on the login form (default on = current behavior).
- On → token in localStorage (survives restart) + username remembered and
  pre-filled next time.
- Off → token in sessionStorage (cleared when the app closes), username not
  remembered.
- checkAuth restores from either storage; logout clears both.

Tests: remember=false uses sessionStorage; username remembered on default;
checkAuth restores a session-only token.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JBdmpwHywhFrDXowDRV52Q
@RebelliousSmile
RebelliousSmile merged commit dc8a3b6 into main Jul 31, 2026
2 checks passed
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.

2 participants