-
Notifications
You must be signed in to change notification settings - Fork 761
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
Remove the "Attempts" column for non-practice quiz resources #13011
Comments
It seems like the original conditionality that should have gated this is still in play: https://github.com/learningequality/kolibri/blob/develop/kolibri/plugins/coach/assets/src/views/common/tables/ReportsLearnersTable.vue#L10 So the data table itself is now having |
This will require some digging through the vuex state to determine how and why this is happening - it appears that it is also happening for non-exercise resource types to - so it is being set to 0 rather than being undefined. |
It may also just be simpler than trying to mess with the passed in |
Hey @rtibbles , I would love to work on this issue. Please assign this to me. |
Hey @you-think-you-know-me! hanks for your interest in contributing to solving this issue! 👐. Unfortunately, this issue is not intended for Open Source contribution. |
The "attempts" column was inadvertently added to all resources in the resource report for learners, when it should only be visible (currently) for practice quizzes. This should only be added for exercises after we complete: #9851
Slack Message
Exercise:
Other resource:
The text was updated successfully, but these errors were encountered: