Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

SSO - Ajouter un encart bloquant de notification de déconnexion lorsque l'utilisateur n'est plus connecté #3759

Open
louptheron opened this issue Oct 10, 2024 · 4 comments · Fixed by #3763 · May be fixed by #3863
Open

SSO - Ajouter un encart bloquant de notification de déconnexion lorsque l'utilisateur n'est plus connecté #3759

louptheron opened this issue Oct 10, 2024 · 4 comments · Fixed by #3763 · May be fixed by #3863
Assignees
@louptheron

Comments

@louptheron
Copy link
Collaborator

louptheron commented Oct 10, 2024 •
edited
Loading

Il arrive qu'un utilisateur laisse MonitorFish en background, le token expire et rend inutilisable l'interface, avec des erreurs 401 en réponse aux APIs.
Il faut rediriger sur /login dans ce cas.
@louptheron louptheron moved this to To do in Board MonitorFish Oct 10, 2024
@louptheron louptheron mentioned this issue Oct 11, 2024
Merged
1 task
@louptheron louptheron moved this from In progress to In Code Review in Board MonitorFish Oct 17, 2024
louptheron added a commit that referenced this issue Oct 17, 2024
@louptheron
SSO - Correction des erreurs de connexion (#3763)
8ff6e6c 
## Linked issues

- Resolve #3759
- Suppression de ApiError pour n'utiliser que FrontendApiError
- Utilisation d'un `<RequireAuth/>`, sur le même principe que sur
MonitorEnv

----

- [ ] Tests E2E (Cypress)
@github-project-automation github-project-automation bot moved this from In Code Review to To deploy in INT for review in Board MonitorFish Oct 17, 2024
@louptheron louptheron moved this from To deploy in INT for review to To review in INT in Board MonitorFish Oct 28, 2024
@louptheron
Copy link
Collaborator Author

@AdelineCelier Je vais faire quelques tests de mon côté. Tu peux aussi tester que la connexion/déconnexion fonctionne bien ?

@louptheron
Copy link
Collaborator Author

PR revert

@louptheron louptheron reopened this Oct 28, 2024
@github-project-automation github-project-automation bot moved this from To review in INT to In progress in Board MonitorFish Oct 28, 2024
@louptheron louptheron moved this from In progress to To deploy in INT for review in Board MonitorFish Oct 28, 2024
louptheron added a commit that referenced this issue Oct 29, 2024
@louptheron
SSO - Correction des erreurs de connexion (#3763)
ec82fe3 
- Resolve #3759
- Suppression de ApiError pour n'utiliser que FrontendApiError
- Utilisation d'un `<RequireAuth/>`, sur le même principe que sur
MonitorEnv

----

- [ ] Tests E2E (Cypress)
@louptheron louptheron moved this from To deploy in INT for review to To review in INT in Board MonitorFish Oct 31, 2024
@louptheron
Copy link
Collaborator Author

@AdelineCelier Je veux bien que tu testes le login/déco et que tu me fasses des retours / proposition sur la page de connexion.

@AdelineCelier
Copy link
Collaborator

AdelineCelier commented Nov 4, 2024 •
edited
Loading

@louptheron comme convenu, qques retours UI sur la page de connexion, que j'ai illustrés sur cet écran :

  • retirer le logo EIG et centrer le texte "MonitorFish" (je l'ai mis en 32 px sur la maquette) au-dessus du bouton "Se connecter avec Cerbère"
  • Pour qu'il soit plus lisible, mettre un fond derrière les textes sous le bouton de la même couleur que le fond bleu de la carte (#88A1BB)
  • Mettre en Bold et 16 px le texte "Vous accédez à une application réservée aux services de l'Etat"
  • Laisser le texte législatif en regular italic 13 px, et faire un retour à la ligne après "Rappels législatifs" + veiller à ce qu'il n'y ait pas de renvois à la ligne de mots oprhelins (cf. en inté aujourd'hui "d'amende".)
  • Laisser le texte "CNSP" en bas en Regular 13 px, lui aussi sur un fond bleu
  • Si possible, reprendre les marges de la maquette entre les blocs et au sein de blocs bleus

@AdelineCelier AdelineCelier reopened this Nov 5, 2024
@github-project-automation github-project-automation bot moved this from To review in INT to In progress in Board MonitorFish Nov 5, 2024
@AdelineCelier AdelineCelier moved this from In progress to To do in Board MonitorFish Nov 5, 2024
@louptheron louptheron linked a pull request Nov 15, 2024 that will close this issue
Open
1 task
@louptheron louptheron moved this from In progress to In Code Review in Board MonitorFish Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@louptheron louptheron

Labels
None yet
Projects
Board MonitorFish
Status: In Code Review
Milestone
No milestone
2 participants
@louptheron @AdelineCelier