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

Afficher message d'alerte sur la publication d'un Indicator ou Operation si l'objet parent (série) est dans l'état Unpublished #795

Closed
CatherineGoillot opened this issue Sep 19, 2024 · 7 comments · Fixed by #794
Assignees

Comments

@CatherineGoillot
Copy link

Reprendre le message de la carte InseeFr/Bauhaus#959.
Vu lors du test sur InseeFr/Bauhaus#743.

@EmmanuelDemey
Copy link
Collaborator

@CatherineGoillot si je regarde bien le code, nous sommes bien d'accord qu'à l'heure actuelle, il n'y aucune verification qui est faite ? Il ne faut pas juste change le message d'erreur, mais bien implémenter cette vérification qui n'existe pas pour le moment ?

@CatherineGoillot
Copy link
Author

J'ai regardé avec Florian.
Pour l'opération, il y a bien une vérification qui est faite par rapport à la série.
Pour l'indicateur, celui-ci peut ne pas être relié à une série donc pas de vérification par rapport à la série.

@EmmanuelDemey
Copy link
Collaborator

Donc nous devons faire le renommage que pour la publication d'une opération @CatherineGoillot @flo7894 ?

@CatherineGoillot
Copy link
Author

CatherineGoillot commented Oct 22, 2024

Après d'âpres discussions, il faudrait que

  • les indicateurs soient rattachés à une série (ce qui n'est pas toujours le cas aujourd'hui) et donc que le message d'alerte soit présent pour la publication d'un indicateur ou d'une opération.

Ce qui veut dire que rajout de :

  • astérisque sur le champ Séries ou indicateurs liés
  • message d'erreur si sauvegarde alors que le champ n'est pas rempli

Veux-tu que je fasse un autre ticket pour cette évolution en lien avec celui-ci ?

@EmmanuelDemey
Copy link
Collaborator

Je pense qu'il est préférable de faire une autre issue.
Dans clle-ci, je ne vais gérer que le problème lors de la publication d'une opération.

@EmmanuelDemey EmmanuelDemey transferred this issue from InseeFr/Bauhaus Oct 25, 2024
@EmmanuelDemey EmmanuelDemey linked a pull request Oct 25, 2024 that will close this issue
@CatherineGoillot
Copy link
Author

OK. Issue faite pour rattachement d'un indicateur à une série (https://github.com/orgs/InseeFr/projects/9/views/5?pane=issue&itemId=85013994&issue=InseeFr%7CBauhaus%7C1053)

@CatherineGoillot
Copy link
Author

Tests : OK
Testé uniquement sur Opération. Message présent quand la série n'a pas été publiée.

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 a pull request may close this issue.

2 participants