Skip to content

Hide Dropout Questionnaire with no results#2610

Merged
janno42 merged 7 commits intoe-valuation:mainfrom
geromequa:2601-hide-dropout-questionnaire
Mar 16, 2026
Merged

Hide Dropout Questionnaire with no results#2610
janno42 merged 7 commits intoe-valuation:mainfrom
geromequa:2601-hide-dropout-questionnaire

Conversation

@geromequa
Copy link
Copy Markdown
Collaborator

Closes #2601

Add simple check in results_evaluation_detail.html and expand test_dropout_results_shown test.

@richardebeling richardebeling requested review from jooooosef and removed request for jooooosef January 19, 2026 19:39
Comment thread evap/results/templates/results_evaluation_detail.html Outdated
Comment thread evap/results/templates/results_evaluation_detail.html Outdated
@geromequa geromequa requested a review from jooooosef February 9, 2026 17:30
Comment thread evap/results/templates/results_evaluation_detail.html Outdated
Comment thread evap/results/templates/results_evaluation_detail.html Outdated
Comment thread evap/results/templates/results_evaluation_detail.html
@geromequa geromequa requested a review from jooooosef February 16, 2026 18:14
Comment thread evap/results/templates/results_evaluation_detail.html Outdated
@geromequa geromequa requested a review from jooooosef February 16, 2026 19:59
Copy link
Copy Markdown
Collaborator

@jooooosef jooooosef left a comment

Choose a reason for hiding this comment

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

jäs sänk ju

Comment thread evap/results/templates/results_evaluation_detail.html Outdated
Copy link
Copy Markdown
Member

@niklasmohrin niklasmohrin left a comment

Choose a reason for hiding this comment

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

Makes sense, thanks!

Comment thread evap/results/tests/test_views.py Outdated
Comment thread evap/results/templates/results_evaluation_detail.html Outdated
@janno42 janno42 merged commit f5fd88c into e-valuation:main Mar 16, 2026
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Dropout questionnaire should only be shown on results page if answers were given

5 participants