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

Préavis – Ajuster les comportements des boutons "enregistrer" et "diffuser" #3750

Open
AdelineCelier opened this issue Oct 8, 2024 · 1 comment · May be fixed by #3874
Open

Préavis – Ajuster les comportements des boutons "enregistrer" et "diffuser" #3750

AdelineCelier opened this issue Oct 8, 2024 · 1 comment · May be fixed by #3874
Assignees
Labels

Comments

@AdelineCelier
Copy link
Collaborator

Comportement observé

  • Lorsque je diffuse un préavis une première fois, le bouton "diffuser" se grise.
  • Si je modifie une première fois le préavis après diffusion, le bouton "diffuser" reste grisé tant que je n'ai pas cliqué sur "enregistrer"
  • Si je modifie le préavis et que je clique sur "enregistrer", le bouton "diffuser" redevient actif – ça c'est ok.
  • En revanche si je modifie une 2è fois le préavis sans l'avoir rediffusé entre temps, le bouton "diffuser" reste actif, quand bien même je n'ai pas enregistré la dernière modification.
  • Par ailleurs, le bouton "enregistrer" reste actif même après que j'aie cliqué dessus, cela n'aide pas à savoir si j'ai bien enregistré ou non les dernières modifications faites.

Comportement attendu

  • Tant qu'il y a une modification du préavis qui n'a pas encore été enregistrée, il faut désactiver le bouton "diffuser"
  • Si toutes les modifications du préavis ont bien été enregistrées, il faut désactiver le bouton "enregistrer", pour montrer que tout est bien enregistré.
@AdelineCelier
Copy link
Collaborator Author

On fait ce ticket une fois que le ticket #3589 est fait, pour appliquer ce comportement aux préavis manuels comme automatiques.

@ivangabriele ivangabriele moved this from In progress to In Code Review in Board MonitorFish Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: In Code Review
Development

Successfully merging a pull request may close this issue.

2 participants