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

🐛 Corrige erreur 500 Quand l'intitulé ou la modalité d'une question est vide #1648

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

cprodhomme
Copy link
Contributor

@cprodhomme cprodhomme commented Sep 2, 2024

Erreur rollbar :
https://app.rollbar.com/a/eva-betagouv/fix/item/eva/960

Vidéo pour démontrer qu'on peut lancer une campagne avec numératie et des questions clic dans image sans intitulé
question 1 : sans intitulé écrit ni audio, avec consigne écrit et audio
question 2 : avec intitulé écrit ni audio, sans consigne écrit et audio
question 3 : sans intitulé écrit avec intitulé audio, sans consigne écrit et audio
question 4 : sans rien sauf intitulé audio
question 5 : sans rien

Enregistrement.de.l.ecran.2024-09-18.a.18.38.25.mov

La démo se fait avec QuestionClicDansImage mais le refacto et les tests permettent de vérifier qu'elle s'applique à tous les types de questions.

@cprodhomme cprodhomme force-pushed the corrige-erreur-500-questions branch from 905dcf5 to 641837b Compare September 4, 2024 16:18
Copy link
Member

@etienneCharignon etienneCharignon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai refactoré pour éviter d'avoir des méthodes de class auxquelles on est obligé de passer une instance de la classe.

app/models/question_clic_dans_image.rb Outdated Show resolved Hide resolved
@marouria marouria force-pushed the corrige-erreur-500-questions branch from db5e981 to fa6ed2c Compare September 18, 2024 09:36
@marouria marouria changed the title 🐛 Corrige erreur 500 sur les questions 🐛 Corrige erreur 500 Quand l'intitulé ou la modalité d'une question est vide Sep 18, 2024
…fixe erreur 500 quand une question n'a pas de transcriptions
@marouria marouria force-pushed the corrige-erreur-500-questions branch from fa6ed2c to 0a59bf9 Compare September 18, 2024 13:36
@etienneCharignon etienneCharignon merged commit addefe7 into develop Sep 18, 2024
5 checks passed
@etienneCharignon etienneCharignon deleted the corrige-erreur-500-questions branch September 18, 2024 14:04
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.

3 participants