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

fix(Télédéclaration): supprime la pré-sélection du mode de saisie de la télédéclaration qui ne s'enregistrait pas #4952

Merged

Conversation

Charline-L
Copy link
Collaborator

@Charline-L Charline-L commented Jan 24, 2025

Quoi ?

En pré-renseignant ce champ, lorsque l'on "passe à l'étape d'après" comme l'app ne détecte pas de changement entre la valeur sélectionnée et la valeur "précédente" elle ne l'enregistre pas.

Du coup réaction en chaine =>

  • dans les TD le champ type de déclaration est null
  • dans la liste des actions le bouton télédéclarer en masse compte les diagnostiques complet et avec un type
  • donc on avait un nombre différent entre le bouton et ce qui est disponible dans le tableau car lui ne fait pas attention au champ

Et surtout cette info était fausse dans notre base !
TD 2024 => 6461 avec un type vide
TD 2025 => déjà 340 avec un type vide

@github-actions github-actions bot requested review from qloridant and raphodn January 24, 2025 11:44
@Charline-L Charline-L changed the title fix(Télédéclaration): enlève la pré-sélection du mode de saisie de la télédéclaration qui ne s'enregistrait pas fix(Télédéclaration): supprime la pré-sélection du mode de saisie de la télédéclaration qui ne s'enregistrait pas Jan 24, 2025
@Charline-L Charline-L merged commit a8d8a08 into staging Jan 27, 2025
12 of 13 checks passed
@Charline-L Charline-L deleted the charline-l/homogeneous-canteen-ready-td-button-and-table branch January 27, 2025 18:01
Copy link
Contributor

You closed this PR and deleted the review app 👋

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.

2 participants