Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.4
    • Component/s: Assignment
    • Labels:
    • Testing Instructions:
      Hide
      1. Go to the "Site Administration" => "Plugins" => "Activity modules" => "Manage activities" page
      2. Delete the "Assignment" plugin
      3. Go to the "Site Administration" => "Notifications" page
      4. Re-install the plugin
      5. Verify the default value for "assignfeedback_comments | default" is "Yes"
      Show
      Go to the "Site Administration" => "Plugins" => "Activity modules" => "Manage activities" page Delete the "Assignment" plugin Go to the "Site Administration" => "Notifications" page Re-install the plugin Verify the default value for "assignfeedback_comments | default" is "Yes"
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull Master Branch:

      Description

      With the old assignment whichever type it was, it was always possible to quick grade with a grade and comment.

      Now with comments put as a sub-type of feedback, this is not the case.

      So if people were allowing a file to be uploaded, having used the old assignment type would they not expect to be able to grade + comment as normal?

      So if the comment aspect has to be maintained as a feedback type (which I don't see the logic in when the comment was part of the grading) the comment feedback should be defaulted to ON in Moodle (right now defaulted to off)

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            marycooch Mary Cooch added a comment - - edited

            yes- I will go with the default to ON -it has caught me out a couple of times until I figured out where to change it myself.

            Show
            marycooch Mary Cooch added a comment - - edited yes- I will go with the default to ON -it has caught me out a couple of times until I figured out where to change it myself.
            Hide
            salvetore Michael de Raadt added a comment -

            Thanks for raising that, Gavin.

            I tend to agree that comments should be on by default. I was planning to raise that myself at one stage, but lost sight of it.

            I suggest we throw this open to the community for some objective opinions.

            Show
            salvetore Michael de Raadt added a comment - Thanks for raising that, Gavin. I tend to agree that comments should be on by default. I was planning to raise that myself at one stage, but lost sight of it. I suggest we throw this open to the community for some objective opinions.
            Hide
            ghenrick gavin henrick added a comment -

            Thanks Michael.

            Must admit I see it as lost functionality right now rather than an improvement, since people need to do something extra just to have what they already had.

            G

            Show
            ghenrick gavin henrick added a comment - Thanks Michael. Must admit I see it as lost functionality right now rather than an improvement, since people need to do something extra just to have what they already had. G
            Hide
            damyon Damyon Wiese added a comment -

            I considered turning the file feedback plugin on by default as well but the file picker takes up quite a lot of room now so I decided to leave it off.

            Show
            damyon Damyon Wiese added a comment - I considered turning the file feedback plugin on by default as well but the file picker takes up quite a lot of room now so I decided to leave it off.
            Hide
            rwijaya Rossiani Wijaya added a comment -

            Patch looks good.

            Thanks Damyon.

            Submitting for integration review.

            Show
            rwijaya Rossiani Wijaya added a comment - Patch looks good. Thanks Damyon. Submitting for integration review.
            Hide
            poltawski Dan Poltawski added a comment -

            The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            TIA and ciao

            Show
            poltawski Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks guys, this has been integrated to master now.

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks guys, this has been integrated to master now.
            Hide
            timb Tim Barker added a comment -

            Congrats, your test passed.

            Show
            timb Tim Barker added a comment - Congrats, your test passed.
            Hide
            nebgor Aparup Banerjee added a comment -

            yay, it works!

            This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week.

            Thank you all for taking the time to get us here.

            cheers!

            Show
            nebgor Aparup Banerjee added a comment - yay, it works! This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week. Thank you all for taking the time to get us here. cheers!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  3/Dec/12