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

feat(sso): add domain configured for sso check before login #1131

Merged
merged 2 commits into from
Oct 16, 2024

Conversation

MarcWadai
Copy link
Contributor

@MarcWadai MarcWadai commented Oct 10, 2024

fixes #1123

Changes

  • Check sso flow available before going further

image

Checklist

  • Tests written for new code (and old code if feasible).
  • New or updated public/exported symbols have accurate TSDoc documentation.
  • Linter and other CI checks pass.
  • Sign-off given on the changes (see CONTRIBUTING.md).

@MarcWadai MarcWadai requested a review from odelcroi October 10, 2024 16:47
@odelcroi
Copy link
Member

@areox-net what do you think

Copy link
Contributor

@areox-net areox-net left a comment

Choose a reason for hiding this comment

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

Je pense qu'on peut changer le message en:
"ProConnect is not yet available for your domain"
"ProConnect n'est pas encore disponible pour votre domaine"

@areox-net
Copy link
Contributor

@odelcroi @MarcWadai J'ai suggéré un changement de phrase en rouge.
Quel action on peut proposer à l'utilisateur? Il peut contacter quelqu'un pour activer l'SSO ?

@areox-net
Copy link
Contributor

En fait, je ne sais pas pour l'user c'est plus clair "pas disponible" ou "desactivé", et si le "pas encore" est utile

"ProConnect is disabled for your domain"
"ProConnect est désactivé pour votre domaine"

@MarcWadai
Copy link
Contributor Author

@areox-net Dans la landing page on utilise le terme administration plutôt que domaine. Je pense que c'est peut être plus clair pour les agents publiques.
Je pense que désactivé est plus juste que disponible, car ce n'est pas forcément un choix technique, mais potentiellement plus l'administration qui ne souhaite pas l'activé . Donc je dirais bien :

"ProConnect is disabled for your administration"
"ProConnect est désactivé pour votre administration"

@MarcWadai MarcWadai force-pushed the 1123-sso-check-hs-configured branch from 644fcb8 to 5625952 Compare October 15, 2024 09:43
@MarcWadai MarcWadai force-pushed the 1123-sso-check-hs-configured branch 2 times, most recently from fe47408 to df5bf42 Compare October 15, 2024 13:30
@areox-net
Copy link
Contributor

@MarcWadai Ok pour "desactivé".
Par contre pour "administration" je suis d'accord avec toi mais dans l'UI on parle que de domaine il me semble. C'est peut-etre une bonne idée de changer mais je propose qu'on le change partout avec un ticket dédié.

@MarcWadai MarcWadai force-pushed the 1123-sso-check-hs-configured branch from df5bf42 to 2dd6836 Compare October 15, 2024 14:37
@MarcWadai
Copy link
Contributor Author

@areox-net c'est ok pour moi ! j'ai fait la modif comme ça (cf capture d'écran)

@MarcWadai MarcWadai requested a review from areox-net October 15, 2024 15:36
@MarcWadai MarcWadai merged commit 17db437 into develop_tchap Oct 16, 2024
7 checks passed
@areox-net
Copy link
Contributor

@MarcWadai Je viens de m'en rendre compte, mais dans le cas de cet erreur le bouton PC ne devrait pas etre actif. On ne peut pas continuer.

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.

PC - login alors que le serveur Synapse n'est pas configuré ProConnect
3 participants