Moodle
  1. Moodle
  2. MDL-30350

Late submissions should be highlighted with late information displayed on the list of submissions page

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.5, 2.1.3, 2.2, 2.3
    • Fix Version/s: 2.1.4, 2.2.1
    • Component/s: Assignment (2.2)
    • Labels:
    • Testing Instructions:
      Hide
      • As a course convenor:
        • create a new assignment
        • set the due date to some time in the past
      • login as an enrolled student
        • submit some work
      • Log back in as the course convenor
        • navigate to the piece of work
        • View the submitted assignments

      Expected Result:
      the submitted assignment should be highlighted in some way to suggest that the work was submitted late

      Actual result:
      The row appears as normal

      Show
      As a course convenor: create a new assignment set the due date to some time in the past login as an enrolled student submit some work Log back in as the course convenor navigate to the piece of work View the submitted assignments Expected Result: the submitted assignment should be highlighted in some way to suggest that the work was submitted late Actual result: The row appears as normal
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-30350-master-1
    • Rank:
      32722

      Description

      When viewing a list of submissions (e.g. http://moodle.local/mod/assignment/submissions.php?id=2), if a piece of work is submitted late:

      • the row should have CSS for theme designers to highlight it
      • it should show how late the work is

        Activity

        Hide
        Michael de Raadt added a comment -

        Hi, Andrew.

        We're bogged down with QA testing and the resulting bug fixing. It may be after this that we have a chance to go through our backlog of external peer reviews.

        Show
        Michael de Raadt added a comment - Hi, Andrew. We're bogged down with QA testing and the resulting bug fixing. It may be after this that we have a chance to go through our backlog of external peer reviews.
        Hide
        Rossiani Wijaya added a comment -

        Hi Andrew,

        Thank you for supplying patch to address the issue.

        The patch looks good and works well.

        Show
        Rossiani Wijaya added a comment - Hi Andrew, Thank you for supplying patch to address the issue. The patch looks good and works well.
        Hide
        Rossiani Wijaya added a comment -

        Submitting for integration review.

        Show
        Rossiani Wijaya added a comment - Submitting for integration review.
        Hide
        Aparup Banerjee added a comment -

        thanks for the patch

        This has been integrated and cherry-picked into master,2.2.x and 2.1.x.

        ready for testing.

        Show
        Aparup Banerjee added a comment - thanks for the patch This has been integrated and cherry-picked into master,2.2.x and 2.1.x. ready for testing.
        Hide
        Rajesh Taneja added a comment -

        Works Great
        Thanks for fixing this, Andrew.

        Show
        Rajesh Taneja added a comment - Works Great Thanks for fixing this, Andrew.
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Yes, you did it!

        Now your code is part of the best weeklies released ever, many thanks!

        Closing, ciao

        Show
        Eloy Lafuente (stronk7) added a comment - Yes, you did it! Now your code is part of the best weeklies released ever, many thanks! Closing, ciao

          People

          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: