-
Notifications
You must be signed in to change notification settings - Fork 5
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
Enhance legacy non migrated token display #1394
Conversation
0bf37c7
to
0dc775b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci @Un3x, on est d'accord que quand on clique ça va ouvrir ça :
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'est quand même compliqué d'un point de vue métier.
Y'a des tests qui traînent pour tester toutes les branches ?
D'ailleurs j'ai l'impression qu'il est possible de ne pas avoir de show du jeton (quasiment sûr btw avec le compact
qui traîne). Pourquoi ?
Et pourquoi on n'affiche pas le show quelque soit l'état du jeton ?
Vu que globalement c'est quasiment que du refacto (modulo le no_action
) j'accepte la PR mais je trouve ça très compliqué en voyant cette PR, imo y'a potentiellement de la simplification à faire d'un point de vue métier.
L'idée de cette colone c'est de pouvoir voir en un clin d'oeil quelles sont les actions en attentes concernant chacune des habilitations et le token. Les actions possibles sont le show et le prolong, mais les labels diffèrent selon la situation. Le cas qui est ajouté dans cette PR (le remplacement du token legacy), disparaîtra en même temps que les tokens legacy. |
Concernant les tests, il y a des tests pour l'affichage du show quand il y a un blacklisting, du prolong et du no action. Il n'y a pas de test dans le cas du show pour une première utilisation, ni pour le cas rajouté ici de changement de jetons legacy (après normalement ce n'est qu'un label qui change) |
1 similar comment
Concernant les tests, il y a des tests pour l'affichage du show quand il y a un blacklisting, du prolong et du no action. Il n'y a pas de test dans le cas du show pour une première utilisation, ni pour le cas rajouté ici de changement de jetons legacy (après normalement ce n'est qu'un label qui change) |
@DorineLam oui c'est ca qu'ils vont voir. Sauf s'ils sont contact métier, auquel cas il y aura le contenu de la popup version contact métier. |
Merci Thomas, ça me semble super. |
0dc775b
to
99a62c4
Compare
No description provided.