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.

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

Add link to status page in the error message when Tchap is down #636

Merged
merged 4 commits into from
Aug 1, 2023

Conversation

odelcroi
Copy link
Member

@odelcroi odelcroi commented Jul 26, 2023

room status bar (UPDATED)

Before :
image

After :
image

Login

Before :
Capture d’écran 2023-07-27 à 10 26 45

After :
Capture d’écran 2023-07-31 à 10 13 30

@odelcroi
Copy link
Member Author

@Caroline-lawson
Copy link

Je suggère le texte suivant:

"La connexion à Tchap n'est pas possible pour le moment. Voir l'état du service
Les messages seront envoyés automatiquement dès que la connexion sera rétablie."

@odelcroi odelcroi requested a review from NicolasBuquet July 27, 2023 08:44
@Caroline-lawson
Copy link

Caroline-lawson commented Jul 27, 2023

Sur l'écran de login, pourquoi on affiche "(err-03)" ?

On pourrait afficher "Voir l'état du service" à la ligne plutôt que à la suite de la phrase précédente ?

Et je pourrais avoir lien vers l'état du service ?

Copy link

@NicolasBuquet NicolasBuquet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dans tchap_translations.json :

  • dans certaines phrases EN, Tchap est écrit "Tchape"
  • dans 2 phrases d'erreur, il est marqué "Voir l'état du service", mais le lien n'est pas donné contrairement aux 3 phrases précédentes.

@Caroline-lawson
Copy link

On peut mettre le lien vers l'état du service ?

@odelcroi
Copy link
Member Author

Merci, dans certains messages, je peux mettre le lien pour qu'il soit cliquable, dans d'autres, je ne peux pas, je dois mettre le lien pour qu'il soit copié-collable

@odelcroi
Copy link
Member Author

odelcroi commented Jul 27, 2023

Sur l'écran de login, pourquoi on affiche "(err-03)" ?

Comme le message est le meme, err-03 me permet de distinguer l'erreur en détail

@Caroline-lawson
Copy link

Il y a souvent beaucoup d'erreurs en cours ? Faudrait voir comment on présente l'état du service.

@NicolasBuquet
Copy link

Si tu corrige les 'Tchape" -> 'Tchap', je te valide la PR dans la foulée.

@areox-net
Copy link
Contributor

Sur l'écran de login, pourquoi on affiche "(err-03)" ?

Comme le message est le meme, err-03 me permet de distinguer l'erreur en détail

Il serait mieux d'avoir cela avant le lien. Avec toute la phrase "Voir l'état du service" en lien bleu.

La connexion à Tchap n'est pas possible pour le moment (Err-03).
Voir l'état du service

@Caroline-lawson
Copy link

et si "voir l'état du service" renvoyait directement à la section de la page concernée ?

Je dis ça mais je ne sais pas quelle tête a la page d'état du service...

@odelcroi
Copy link
Member Author

la page de l'état du service : https://status.tchap.numerique.gouv.fr

@areox-net
Copy link
Contributor

C'est très bien.
Dans la page login, dommage pour l'aller à la ligne sur le message en rouge pour très peu de charactères.

@Caroline-lawson Ca serait choquant d'avoir un message plus court avec une phrase sans article et verbe ? Genre, "Connexion à Tchap impossible pour le moment."
ou
"Connexion à Tchap momentanément impossible."

@mcalinghee mcalinghee closed this Aug 1, 2023
@mcalinghee mcalinghee reopened this Aug 1, 2023
@mcalinghee
Copy link
Contributor

Test - OK
Juste un message "Your Tchap is misconfigured" mais non bloquant :
image

@mcalinghee mcalinghee dismissed NicolasBuquet’s stale review August 1, 2023 13:38

correction were implemented

@odelcroi odelcroi changed the title Change error message when tchap service is down Add link to status page in the error message when Tchap is down Aug 1, 2023
@odelcroi odelcroi merged commit 50d7c8d into develop_tchap Aug 1, 2023
@Caroline-lawson
Copy link

C'est très bien. Dans la page login, dommage pour l'aller à la ligne sur le message en rouge pour très peu de charactères.

@Caroline-lawson Ca serait choquant d'avoir un message plus court avec une phrase sans article et verbe ? Genre, "Connexion à Tchap impossible pour le moment." ou "Connexion à Tchap momentanément impossible."

Ah oui c'est bien ça

@Caroline-lawson
Copy link

Test - OK Juste un message "Your Tchap is misconfigured" mais non bloquant : image

On peut supprimer le texte en anglais ?

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.

Quand les services ne sont pas accessibles, j'aimerais qu'on m'indique quoi faire
5 participants