search

UNIMOODLE / moodle-local_coursetransfer

Lote P3.2 - Restauración de cursos entre plataformas
2 stars 2 forks source link

Functionality 5. Teacher plug-in to restore teacher courses between platforms controlled by a moodle capabilities set #7

Closed mapemo4 closed 4 months ago

mapemo4 commented 7 months ago

This functionality as a teacher:

5.1 Both moodles must have the same source and destination settings. Try to hide the source teacher from the target moodle, as it may be misleading, you could add a parameter. 5.2 Restoring a category at destination is done correctly, but it is labelled as a 'New category' when it should have the same name and ID number as the source category.

xpazv commented 7 months ago

In review

xpazv commented 7 months ago

5.2 solved in last version uploaded.

xpazv commented 7 months ago

5.1 @mapemo4 could you please give us an example or more details about the requirement?

mapemo4 commented 7 months ago

5.1 @mapemo4 could you please give us an example or more details about the requirement?

Regarding 5.1, it is really a usability problem. The teacher attempts to access the restore remote course menu, but it appears on the drop-down list two Moodles: the origin Moodle (remote) and the Moodle destination (local). However, the original Moodle should only be displayed (remote). For the teacher role, I don't see any sense that he could choose as origin the Moodle destination (local). This Moodle is the starting point for the action.

xpazv commented 7 months ago

As the documentation specifies, you can include any sources and destinies, in the sites list settings

It is not mandatory, for the use of the plugin, that all Moodles involved in the course transfer process, are registered in both settings (sources and destinies) via admin/settings.php?section=local_coursetransfer throough destiny_sites and origin_sites settings

An example:

Two Moodles (3.11 and 4.1) both connected. Each one can request coursse to the other and each one can provide courses to the other

3.11 destinity sites setting, where we include the 4.11 site as remote cs_311 3.11 origin sites setting, where we include the 4.11 site as remote cs_311_origin

Dropdown menu when requesting courses fro 3.11 to 4.1 cs_311_restore_cat

4.1 destinity sites setting, where we include the 3.11 site as remote cs_411

4.1 origin sites setting, where we include the 3.1 site as remote cs_411_origin

Dropdown menu when requesting courses from 4.1 to 3.11 cs_411_remote_restore

mapemo4 commented 6 months ago

As the documentation specifies, you can include any sources and destinies, in the sites list settings

It is not mandatory, for the use of the plugin, that all Moodles involved in the course transfer process, are registered in both settings (sources and destinies) via admin/settings.php?section=local_coursetransfer throough destiny_sites and origin_sites settings

An example:

Two Moodles (3.11 and 4.1) both connected. Each one can request coursse to the other and each one can provide courses to the other

3.11 destinity sites setting, where we include the 4.11 site as remote cs_311 3.11 origin sites setting, where we include the 4.11 site as remote cs_311_origin

Dropdown menu when requesting courses fro 3.11 to 4.1 cs_311_restore_cat

4.1 destinity sites setting, where we include the 3.11 site as remote cs_411

4.1 origin sites setting, where we include the 3.1 site as remote cs_411_origin

Dropdown menu when requesting courses from 4.1 to 3.11 cs_411_remote_restore

It works correctly, you're right. Putting the remote Moodle at the origin and selecting the local one is not always necessary. We have removed the local, and it seems to be working correctly.

mapemo4 commented 6 months ago

It works correctly, you're right. Putting the remote Moodle at the origin and selecting the local one is not always necessary. We have removed the local, and it seems to be working correctly.

xpazv commented 6 months ago

Can we close the issue then, as both points have been solved and clarified?

mapemo4 commented 6 months ago

Please, could you give us a screenshot providing the changes applied?. We dont see the changes yet. 5.2 Restoring a category at destination is done correctly, but it is labelled as a 'New category' when it should have the same name and ID number as the source category.

mapemo4 commented 5 months ago

@xpazv There is a new permission to review as a teacher role: -Teacher capabilities The permissions/capabilities issue is related to the courses a teacher should see. We attach this screenshot with the capabilities. A teacher should not see all the origin courses, so a teacher should only see the courses in which he/she is enrolled as a teacher: teacher_capabilities_set

Steps to reproduce: Home More> Restore Remote Course> Restore Remote Course >To restore a course from another platform, you must first select the site where the original course is located.

Then Check Permissions if the teacher is enrolled in these courses.

Steps to reproduce: Home More> Restore Remote Course> Restore Remote Course >To restore a course from another platform, you must first select the site where the original course is located. Click Next. > Restore Remote Course. A screen shows Restore Remote Course - Select the course to restore.

Actual result:

Now, a screen appears with a list of available courses.

It seems to be the origin list courses

Expected result

Check Permissions if the teacher is enrolled in these courses and show these courses in this screen

xpazv commented 4 months ago

Under review. Will be fixed in next version uploaded