Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: moodle.org
    • Labels:
      None

      Description

      This is similar to MDL-31437, but with some specifics for moodle.org.

      We need a plugin which will:

      • Sync cohorts with course groups across all courses
      • But only add the user to the course group if they are also enrolled in the course

      Example use case:

      • Eloy is a core developer, he is enrolled intwo courses English Using Moodle, and Spanish Using Moodle.
      • Eloy is a member of two system context Cohorts, 'Core Developers' and 'Moodle HQ'.
      • Eloy should be added to core developers and Moodle HQ groups on both the english and spanish course.
      • Eloy SHOULD NOT be enrolled into the Thai using moodle course
      • When eloy enrols onto the Thai using moodle course, he should be added to the 'Core Developers' and 'Moodle HQ' groups.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            poltawski Dan Poltawski added a comment -

            Assigning this one to you Petr, as I think you've agreed to take this one.

            Show
            poltawski Dan Poltawski added a comment - Assigning this one to you Petr, as I think you've agreed to take this one.
            Hide
            tsala Helen Foster added a comment -

            Regarding the example use case, I'm not sure whether we want Particularly Helpful Moodlers to be a cohort. I think it should be a group, as it is now, with the name translated into other languages e.g. Moodlers de gran ayuda in Moodle en Español. Otherwise, multi-talented people like Eloy and Mary will end up with tons of PHM group icons from different courses!

            Show
            tsala Helen Foster added a comment - Regarding the example use case, I'm not sure whether we want Particularly Helpful Moodlers to be a cohort. I think it should be a group, as it is now, with the name translated into other languages e.g. Moodlers de gran ayuda in Moodle en Español. Otherwise, multi-talented people like Eloy and Mary will end up with tons of PHM group icons from different courses!
            Hide
            poltawski Dan Poltawski added a comment -

            Helen, they will all end up as groups in the course (in order to show the forum icon).

            But are you saying you don't think the PHM group should be global? I thought that we discussed wanting to encourage that, so that Mary will be identified as a helpful moodler when she first starts posting in the italian course, rather than just a 'nobody'?

            Show
            poltawski Dan Poltawski added a comment - Helen, they will all end up as groups in the course (in order to show the forum icon). But are you saying you don't think the PHM group should be global? I thought that we discussed wanting to encourage that, so that Mary will be identified as a helpful moodler when she first starts posting in the italian course, rather than just a 'nobody'?
            Hide
            tsala Helen Foster added a comment -

            I remember we talked about it, but don't remember deciding upon it. I just think lots of PHM group icons will look weird. Also, if someone is a PHM in English say, then starts posting in another course, I don't think it will take them long to become a PHM there too.

            Show
            tsala Helen Foster added a comment - I remember we talked about it, but don't remember deciding upon it. I just think lots of PHM group icons will look weird. Also, if someone is a PHM in English say, then starts posting in another course, I don't think it will take them long to become a PHM there too.
            Hide
            poltawski Dan Poltawski added a comment -

            Hi Helen, I updated the description to what I think we are agreed on..

            Show
            poltawski Dan Poltawski added a comment - Hi Helen, I updated the description to what I think we are agreed on..
            Hide
            skodak Petr Skoda added a comment -

            Thanks for the description, it is clear to me what is requested. I am going to work on this before the hackfest.

            Show
            skodak Petr Skoda added a comment - Thanks for the description, it is clear to me what is requested. I am going to work on this before the hackfest.
            Hide
            skodak Petr Skoda added a comment -

            Here is the plugin https://github.com/skodak/moodle-enrol_groupsync, it will be probably moved to hq repo later. Thanks for the report.

            Show
            skodak Petr Skoda added a comment - Here is the plugin https://github.com/skodak/moodle-enrol_groupsync , it will be probably moved to hq repo later. Thanks for the report.
            Hide
            skodak Petr Skoda added a comment -

            Reopening and reassigning ,this need to be installed on moodle.org once we switch to 2.4 branch there.

            Show
            skodak Petr Skoda added a comment - Reopening and reassigning ,this need to be installed on moodle.org once we switch to 2.4 branch there.
            Hide
            dougiamas Martin Dougiamas added a comment -

            Thanks for writing the code, Petr.

            Dan, all yours to test and install on moodle.org.

            Show
            dougiamas Martin Dougiamas added a comment - Thanks for writing the code, Petr. Dan, all yours to test and install on moodle.org.
            Hide
            poltawski Dan Poltawski added a comment -

            Upgrading clone to 2.4, and gonna test this out.

            Show
            poltawski Dan Poltawski added a comment - Upgrading clone to 2.4, and gonna test this out.
            Hide
            poltawski Dan Poltawski added a comment -

            Looking good right now.

            Show
            poltawski Dan Poltawski added a comment - Looking good right now.
            Hide
            tsala Helen Foster added a comment -

            Dan, can we install it on moodle.org, or should I test it more on the clone first?

            Show
            tsala Helen Foster added a comment - Dan, can we install it on moodle.org, or should I test it more on the clone first?
            Hide
            poltawski Dan Poltawski added a comment -

            Tested on the clone and now installed on moodle.org

            Show
            poltawski Dan Poltawski added a comment - Tested on the clone and now installed on moodle.org
            Hide
            skodak Petr Skoda added a comment -

            Could you please clone my repo to the moodlehq organisation on github? Should this be submitted to the plugins database? Who would be the maintainer there?

            Show
            skodak Petr Skoda added a comment - Could you please clone my repo to the moodlehq organisation on github? Should this be submitted to the plugins database? Who would be the maintainer there?
            Hide
            tsala Helen Foster added a comment -

            Petr, thanks a lot for the cohorts-groups sync plugin which is working perfectly on moodle.org.

            Show
            tsala Helen Foster added a comment - Petr, thanks a lot for the cohorts-groups sync plugin which is working perfectly on moodle.org.
            Hide
            poltawski Dan Poltawski added a comment -

            Petr: i've done that and given you access to it.

            Seems like a useful addition to the plugins DB to me - surely you should be the maintainer

            Show
            poltawski Dan Poltawski added a comment - Petr: i've done that and given you access to it. Seems like a useful addition to the plugins DB to me - surely you should be the maintainer
            Hide
            skodak Petr Skoda added a comment -

            Here it is https://moodle.org/plugins/view.php?plugin=enrol_groupsync, I needed to bump up the version to make the plugins database happy, I have also created a new branch MOODLE_24_STABLE that should be used by moodle.org server, ciao.

            Show
            skodak Petr Skoda added a comment - Here it is https://moodle.org/plugins/view.php?plugin=enrol_groupsync , I needed to bump up the version to make the plugins database happy, I have also created a new branch MOODLE_24_STABLE that should be used by moodle.org server, ciao.
            Hide
            poltawski Dan Poltawski added a comment -

            Thanks Petr, switched to the 24_STABLE branch.

            Show
            poltawski Dan Poltawski added a comment - Thanks Petr, switched to the 24_STABLE branch.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development