- Issue created by @Graber
- Merge request !98#3526611: Add a global setting to conditionally allow ungraded course access. → (Merged) created by Graber
- 🇬🇧United Kingdom catch
Couple of questions on the MR - it's not really clear to me why 'revisit mode' doesn't already handle this - e.g. if you can revisit after a course is graded, I think you would be able to revisit before? But last time I tested that, it looked like that wasn't possible. I think if we changed that, then we might not need the setting at all?
If revisit mode is off, then stopping people from going back in until grading is complete seems appropriate still, although agreed that if we need this level of granularity, then a global setting probably makes sense here.
- 🇵🇱Poland Graber
if you can revisit after a course is graded, I think you would be able to revisit before?
We don't know what different requirements will we encounter in the future.
I'd expect this or that client will not want people to revisit until fully graded (let's say it's an exam with some open questions - editing answers after finishing and before everything is graded would defeat the purpose). - 🇬🇧United Kingdom catch
I'd expect this or that client will not want people to revisit until fully graded (let's say it's an exam with some open questions - editing answers after finishing and before everything is graded would defeat the purpose).
Oh hmm I had not thought of that. So allow users to go back and see what they did after things are graded, but not to do it until then.
- 🇬🇧United Kingdom catch
Gave this a quick manual test locally and it worked for me.
I think we should create a follow-up to add phpunit coverage for this - might be possible to add a couple of lines to the existing coverage even.
- 🇵🇱Poland Graber
Thanks, created 📌 Extend course navigation test coverage Active as a follow-up.