Elabore/survey-tools
8
0
Fork 0
Code Issues Pull Requests Actions Packages Releases Activity

[IMP] survey_record_generation: first refacto of record creation #8

Merged
mondot merged 1 commits from survey_record_generation-refacto into 16.0 2025-10-23 16:22:48 +00:00
Conversation 2 Commits 1 Files Changed 5 +440 -146
mondot commented 2025-10-16 14:46:15 +00:00
Owner
Copy Link

TODO :

  • Check code coverage (we goes from 88% to 86% because I've added some UserError, not tested)
  • Check that it did not brake something for cca44 and calim
  • Add translations
TODO : - [x] Check code coverage (we goes from 88% to 86% because I've added some UserError, not tested) - [x] Check that it did not brake something for cca44 and calim - [x] Add translations
mondot changed target branch from survey_record_generation-tests-and-refacto-for-survey-record-creation to 16.0 2025-10-20 14:12:53 +00:00
mondot added 1 commit 2025-10-20 14:12:53 +00:00
[IMP] survey_record_generation: first refacto of record creation 159e1dd05e
stephansainleger approved these changes 2025-10-20 14:52:25 +00:00
mondot added 2 commits 2025-10-20 15:35:16 +00:00
[IMP] survey_record_generation: first refacto of record creation 159e1dd05e
[IMP] survey_record_generation: first refacto of record creation
Some checks failed
pre-commit / pre-commit (pull_request) Failing after 1m34s
Details
a03c4ae4e7
mondot commented 2025-10-21 07:27:19 +00:00
Author
Owner
Copy Link

To check before merging :

  • fresque océane
  • le lien => le fonctionnement est le même entre l'ancien code et le code refacto (les sondages ne sont pas très bien configurés)
  • archijeux => n'utilise pas le module survey_record_generation

Bugs détectés corrigés :

  • Pour la fresque océane : [Survey record generation] La valeur no pour le champs 'answer_values_type' n'est pas supportée (pour la question Situation anims). qui ne pose pas de problème sur la branche 16.0. En fait ce n'est pas un bug mais un problème qui était mis sous silence avant la refacto. J'ai reformulé la traduction pour préciser qu'il faut utiliser 'Enregistrement' ou 'Valeur'. En parallèle j'ai vérifié leur sondage en prod.
To check before merging : - [x] fresque océane - [x] le lien => le fonctionnement est le même entre l'ancien code et le code refacto (les sondages ne sont pas très bien configurés) - [x] archijeux => n'utilise pas le module survey_record_generation Bugs détectés corrigés : - [x] Pour la fresque océane : `[Survey record generation] La valeur no pour le champs 'answer_values_type' n'est pas supportée (pour la question Situation anims).` qui ne pose pas de problème sur la branche 16.0. En fait ce n'est pas un bug mais un problème qui était mis sous silence avant la refacto. J'ai reformulé la traduction pour préciser qu'il faut utiliser 'Enregistrement' ou 'Valeur'. En parallèle j'ai vérifié leur sondage en prod.
mondot added 2 commits 2025-10-23 13:06:11 +00:00
[IMP] survey_record_generation: first refacto of record creation
Some checks failed
pre-commit / pre-commit (pull_request) Failing after 1m34s
Details
a03c4ae4e7
[IMP] survey_record_generation: first refacto of record creation
Some checks failed
pre-commit / pre-commit (pull_request) Failing after 1m33s
Details
fd25c2b970
mondot merged commit fd25c2b970 into 16.0 2025-10-23 16:22:48 +00:00
mondot deleted branch survey_record_generation-refacto 2025-10-23 16:22:48 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
stephansainleger
Labels
Clear labels
bug
Something isn't working
documentation
Improvements or additions to documentation
duplicate
This issue or pull request already exists
enhancement
New feature or request
good first issue
Good for newcomers
help wanted
Extra attention is needed
invalid
This doesn't seem right
question
Further information is requested
wontfix
This will not be worked on
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
admin borisgallet elabore_bot laetitiadacosta mathieusarrasin mondot stephansainleger vlab
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Elabore/survey-tools#8
No description provided.
Delete Branch "survey_record_generation-refacto"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?