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

[TECH] Supprimer toutes les références au statut "partially" (PIX-11417) #8937

Merged
merged 3 commits into from
May 22, 2024

Conversation

La-toile-cosmique
Copy link
Contributor

@La-toile-cosmique La-toile-cosmique commented May 15, 2024

🦄 Problème

Autrefois les "answers" pouvaient avoir un "result" égal à "partially". Ce type de résultat n'est plus utilisé depuis au moins 2021. Cependant le code référence et prévoit toujours ce type de résultat.

🤖 Proposition

Oblitérer toute référence à ce "result" dans le code.

💯 Pour tester

Tests au vert

@La-toile-cosmique La-toile-cosmique added the team-evaluation PR relatives à l'expérience d'évaluation label May 15, 2024
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@La-toile-cosmique La-toile-cosmique force-pushed the pix-11417-remove-partially-result branch from 65b3d1c to 3a9a769 Compare May 15, 2024 15:06
@La-toile-cosmique La-toile-cosmique changed the title 11417 [TECH] Supprimer toutes les références au statut "partially" (PIX-11417) May 21, 2024
@La-toile-cosmique La-toile-cosmique force-pushed the pix-11417-remove-partially-result branch 2 times, most recently from d0715e4 to 44fb9c6 Compare May 21, 2024 09:34
@La-toile-cosmique La-toile-cosmique marked this pull request as ready for review May 21, 2024 09:44
@La-toile-cosmique La-toile-cosmique force-pushed the pix-11417-remove-partially-result branch 2 times, most recently from 604fe84 to d235241 Compare May 21, 2024 14:08
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@pix-service-auto-merge pix-service-auto-merge merged commit 60cf83f into dev May 22, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-11417-remove-partially-result branch May 22, 2024 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 Ready to Merge team-evaluation PR relatives à l'expérience d'évaluation Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants