Only show attempts column in lesson reports for practice quizzes #13046
Conversation
github-actions
bot
added
APP: Coach
| anyTries() { | ||
| return this.entries.some(entry => !isUndefined(entry.statusObj.tries)); | ||
| }, | ||
| // Presumes this is a Lesson report table as quizId is only present in quiz routes |
There was a problem hiding this comment.
This got me a bit confused initially, thought this might help future-me (or someone else) because I was trying to use this.exam to get the currently-viewed resource and didn't realize the need to handle varying route params
There was a problem hiding this comment.
So, what you're doing as your approach is using a combination of the modality and the params to find where you are. If there's an exerciseId, you're in a lesson report table, and if it's a quizId, the same exercise is a "quiz". And if you're in a lesson, check the modality to see if it's a practice quiz, in which case show the tries, otherwise, nothing. Right?
Build Artifacts
|
There was a problem hiding this comment.
Thanks @nucleogenesis - I confirm that this is implemented as specified above.
nucleogenesis
deleted the
13011-remove-attempts-column-non-practice-quiz
branch
Summary
Modifies the condition in the ReportsLearnersTable for showing the attempts column so that it only shows if the currently viewed piece of content is a practice quiz.
Only affects the case in which the report is viewing a particular piece of content in a lesson.
References
Fixes #13011
Reviewer guidance
Edge case: When you have a practice quiz but no attempts then in this case there should be no attempts column.