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: duration components in resizing #1159

Merged
merged 3 commits into from
Nov 18, 2024

Conversation

nsenave
Copy link
Contributor

@nsenave nsenave commented Nov 18, 2024

Summary

Done

Les composants de type DURATION étaient ignorés dans la méthode des LunaticUtils qui va chercher la liste des réponses d'une boucle.

L'erreur était silencieuse car le cas default dans le switch d'une des méthodes se contentait d'un log.debug 😶


NB : Les composants de type SUGGESTER étaient ignorés aussi. On avait donc le même bug pour les suggester saisis depuis Pogues dans une boucle. Mais ça ne concerne pas les suggester créés avec un traitement spécifique (le cas le plus utilisé aujourd'hui encore j'imagine).

C'est corrigé au passage.

@nsenave nsenave self-assigned this Nov 18, 2024
@nsenave nsenave added the deploy-snapshot To be used in PR to trigger snapshot deploy pipeline label Nov 18, 2024
@nsenave nsenave marked this pull request as draft November 18, 2024 10:33
@nsenave
Copy link
Contributor Author

nsenave commented Nov 18, 2024

des corrections à faire à la marge encore

@nsenave nsenave marked this pull request as ready for review November 18, 2024 11:44
@nsenave nsenave added deploy-snapshot To be used in PR to trigger snapshot deploy pipeline and removed deploy-snapshot To be used in PR to trigger snapshot deploy pipeline labels Nov 18, 2024
Copy link

👋 Version 3.29.1-SNAPSHOT deployed on docker hub

@github-actions github-actions bot removed the deploy-snapshot To be used in PR to trigger snapshot deploy pipeline label Nov 18, 2024
@nsenave nsenave merged commit 03d0828 into v3-next Nov 18, 2024
@nsenave nsenave deleted the fix/duration-components-in-resizing branch November 18, 2024 15:32
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