Remove feature flag for channel quizzes, thus fixing completion/duration dropdowns#3722
Conversation
…ration options interactions
rtibbles
left a comment
There was a problem hiding this comment.
Checked with both admin and non-admin users. Only slight possible issue here is users not understanding what a practice quiz is - but we can address that later.
|
Hi @marcellamaki, when creating an exercise if I go ahead and select Practice quiz in the Completion drop-down without having specified the Goal value beforehand then the time values in minutes are not being displayed at all for any of the Duration options and it complains that the resource is incomplete. I have to set the Goal value in order to get the Minutes drop-down displayed: 2022-10-19_15-42-30.mp4I've filed separately a possibly related issue: #3751 |
|
Thanks @pcenov -- I'll take a look at this, and then take a look at the other issue you mention as well! |
|
My testing with completion on exercises was also bumpy, but may be related to this, so I'll expose it here unless you prefer a completely new issue.
Very confusing and inconsistent, but my main qualm is the presence of the Duration field, as to me completion of type When goal is met goes against setting a duration for the resource... 🤔 It's either goal accomplished by the number of required correct answers in an exercise, or by duration... During which the number of correct answers does not count? completion-objective-reached.mp4 |
Summary
Description of the change(s) you made
This PR removes the channel quiz/practice quiz feature flag, based on a conversation with Richard. This feature was causing complications in the edit modal around completion and duration options.
Manual verification steps performed
Reviewer guidance
How can a reviewer test these changes?
See above
Comments
Are the of the right UI options available for all exercises now? 🤔 I'm not actually 100% sure
References
#3705 (comment)
Contributor's Checklist
PR process:
CHANGELOGlabel been added to this PR. Note: items with this label will be added to the CHANGELOG at a later timedocslabel has been added if this introduces a change that needs to be updated in the user docs?requirements.txtfiles also included in this PRStudio-specifc:
notranslateclass been added to elements that shouldn't be translated by Google Chrome's automatic translation feature (e.g. icons, user-generated text)pages,components, andlayoutsdirectories as described in the docsTesting:
Reviewer's Checklist
This section is for reviewers to fill out.
yarnandpip)