Skip to content

Add option to hide correct answer from feedback - #334

Merged
PhilippImhof merged 5 commits into
FormulasQuestion:mainfrom
PhilippImhof:general-noanswer
Aug 2, 2026
Merged

Add option to hide correct answer from feedback#334
PhilippImhof merged 5 commits into
FormulasQuestion:mainfrom
PhilippImhof:general-noanswer

Conversation

@PhilippImhof

Copy link
Copy Markdown
Collaborator

Fixes #242

@coveralls

coveralls commented Jul 31, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 30758937147

Coverage increased (+0.009%) to 97.5%

Details

  • Coverage increased (+0.009%) from the base build.
  • Patch coverage: 19 of 19 lines across 5 files are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 4760
Covered Lines: 4641
Line Coverage: 97.5%
Coverage Strength: 1145.58 hits per line

💛 - Coveralls

@PhilippImhof

Copy link
Copy Markdown
Collaborator Author

@FloMiLe: If you have some time, I'd love to get your feedback on this.

The option is placed below the part's "General feedback", because IMHO its use only makes sense in combination with a general feedback that gives the student the correct answer in some appropriate way.

@FloMiLe

FloMiLe commented Aug 2, 2026

Copy link
Copy Markdown

@PhilippImhof: That is really great news! I look forward to testing this feature.

Sorry for asking again, though: Last time you provided a download link for the installable zip file including the new commits. Can I somehow obtain that zip file myself?

@PhilippImhof

Copy link
Copy Markdown
Collaborator Author

Oh yes, sorry, I forgot that. You should be able to get it from here:

https://github.com/PhilippImhof/moodle-qtype_formulas/tree/general-noanswer

I hope it works, it's the first time I try to copy such a link in the app...

@FloMiLe

FloMiLe commented Aug 2, 2026

Copy link
Copy Markdown

Thanks for the link! I hope I'll remember how to find my way to the relevant branch myself next time. (I've noticed that it's actually linked at the top of the page...)

And of course many thanks for implementing this feature. I've tested it on some of my questions, and can confirm that it works flawlessly so far. The placement of the check mark is ideal, and the tool tip is very helpful. 🙏 🎉

Just to mention a potential for a possible confusion: The term "general feedback" occurs already as the global "General feedback" for the entire question. So you could consider changing the check mark's description to "Do not show the correct answer along with the part's general feedback." if you agree and that wouldn't contradict the nomenclature in other places.

@PhilippImhof

Copy link
Copy Markdown
Collaborator Author

Just to mention a potential for a possible confusion: The term "general feedback" occurs already as the global "General feedback" for the entire question. So you could consider changing the check mark's description to "Do not show the correct answer along with the part's general feedback." if you agree and that wouldn't contradict the nomenclature in other places.

Thanks, that's a good point. I will change the label accordingly.

@PhilippImhof
PhilippImhof merged commit 4bc9bc9 into FormulasQuestion:main Aug 2, 2026
41 of 42 checks passed
@PhilippImhof
PhilippImhof deleted the general-noanswer branch August 2, 2026 19:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Option to not show the correct answer in a part's general feedback

3 participants