-
Notifications
You must be signed in to change notification settings - Fork 2
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
Init a tab for instructors to visualise habilitations #655
Init a tab for instructors to visualise habilitations #655
Conversation
un quick win est de faire des liens vers les habilitations depuis l'historique, je trouve ça un poill overkill de faire un onglet (alors qu'en vrai... c'est surtout important pour le demandeur et non l'instructeur) |
C'est important pour les instructeurs DGFIP de voir les habilitations sandbox & prod d'une demande. Je vais aussi faire le lien dans l'historique je pense. |
Pour le demandeur, lui montrer l'historique c'est un peu overkill je pense. Imo on doit revoir la présentation des demandes et des habilitations pour les séparer, et ça suffira à clarifier le tout. Les demandes draft, en cours et refusées dans une liste, et les habilitations validées et révoquées dans une autre. |
Lien qui existe dans l'historique de la demande côté instructeur.
Je n'ai pas d'avis sur l'historique entier, par contre pour le demandeur ça me semble pas déconnant d'avoir un listing des habilitations validées (pour voir les anciennes versions).
imo ça a de la valeur si l'organisation a > 1 demande, séparer les 2 notions je ne suis pas sûr que ça clarifie le tout, je pense même que ça aura l'effet inverse. Après, l'index des demandes/habilitations c'est à mon sens peu important, 99% des gens vont venir des notifications et donc arriver sur les demandes direct (ou tout du moins on doit pousser vers ça). tl;dr:
|
Non ? |
Oui faut le mettre, mais c'est déjà mentionné, suffit d'ajouter le lien justement (ce qui est faisable facilement vs refaire une page entière). |
Si dans la même page y'a écrit "Liste de mes demandes" et en dessous "Liste de mes habilitations", et que dans le cas où une liste est vide on affiche pas la section, je pense que ça sera pas confusant. |
Je pense que ça vaut le coup d'avoir une page qui résume simplement la liste des habilitations, versus les avoir noyées dans un historique d'allers retours qui peut être long, surtout dans les cas de la dgfip. |
as u ouiche, c'est je pense du détail pour les raisons évoquées plus haut (cf lien)
A/R qui n'existent pas vraiment vu qu'ils refusent et ne demande pas de modifications 😅 btw, sur le listing des habilitations, tu vas mettre toutes les habilitations, ou seulement la dernière ? pour une habilitation donnée tu vas omettre la demande dans la liste ? si non, tu vas mettre les 2 ? |
4d17c7b
to
ebbcd59
Compare
Let me cook for a while please, j'ai pas fini c'est juste un draft, on reprend la conversation lorsque je la passerait en PR 🙏 |
74893bb
to
3070820
Compare
Je m'occuperai de faire le display équivalent pour les demandeurs dans une PR followup, puis j'ajouterai carrément la séparation plus claire des demandes et habilitations dans l'UI dans une autre PR ensuite. |
(le "durant la demande N" est pas clair) |
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.
Y'a des petits trucs/questions mais globalement c'est OK
app/views/authorization_requests/shared/_reopening_callout.html.erb
Outdated
Show resolved
Hide resolved
app/views/instruction/authorization_request_events/_authorization_request_event.html.erb
Show resolved
Hide resolved
60e71a3
to
c1f882c
Compare
626a3ff
to
606da2b
Compare
7fbb745
to
dff5a7f
Compare
dff5a7f
to
c173f81
Compare
c173f81
to
ca16c4a
Compare
Closes https://linear.app/pole-api/issue/API2-340/afficher-les-habilitations-dune-demande