Optionally allow access to ungraded courses

Created on 26 May 2025, 10 days ago

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Feature request
Status

Active

Version

1.0

Component

Courses and lessons

Created by

🇵🇱Poland Graber

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue 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.

  • Pipeline finished with Skipped
    7 days ago
    #509242
    • graber committed 66566c1e on 1.0.x
      Issue #3526611: Optionally allow access to ungraded courses
      
  • 🇵🇱Poland Graber

    Thanks, created 📌 Extend course navigation test coverage Active as a follow-up.

Production build 0.71.5 2024