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

Quiz creation: Show correct answers for questions #13038

Conversation

nucleogenesis
Copy link
Member

@nucleogenesis nucleogenesis commented Jan 29, 2025

Summary

Puts the showCorrectAnswers=true prop onto the ContentRenderer in QuestiosnAccordion.

I also applied it in ReplaceQuestions, although that is liable to be removed.

Also, in the QuestionListPreview it is explicitly set to false (which I did, but I don't recall why) - but should I also flip that one?

References

Fixes #13025

Reviewer guidance

Do you see correct answers in Quiz Creation after adding questions?

@github-actions github-actions bot added APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) DEV: frontend labels Jan 29, 2025
@nucleogenesis nucleogenesis force-pushed the 13025-correct-answers-missing-in-coach branch from 985f1ce to 105626a Compare January 29, 2025 00:28
@marcellamaki
Copy link
Member

Also, in the QuestionListPreview it is explicitly set to false (which I did, but I don't recall why) - but should I also flip that one?

my inclination is yes - anywhere in coach the correct answer should be indicated

Copy link
Member

@marcellamaki marcellamaki left a comment

Choose a reason for hiding this comment

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

Hey @nucleogenesis - testing this out manually it doesn't seem to resolve the problem. I'm not sure if there's some underlying bug with the showCorrectAnswer prop or with the perseus renderer, but I think this quick fix may unfortunately have been too good to be true :(

Screenshot 2025-01-31 at 10 23 54 AM

Screenshot 2025-01-31 at 10 34 04 AM

Copy link
Member

@marcellamaki marcellamaki left a comment

Choose a reason for hiding this comment

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

thanks @nucleogenesis - the typo fix resolved!

@marcellamaki marcellamaki merged commit b205f4d into learningequality:develop Jan 31, 2025
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) DEV: frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Correct answers not visible while coaches are browsing questions to add to a quiz
2 participants