Syxton / moodle-tool_coursearchiver

GNU General Public License v3.0
7 stars 9 forks source link

Add category-path to course name in user email #61

Open rsacher1 opened 3 months ago

rsacher1 commented 3 months ago

over years many users create courses in different categories (eg 2020, 2021, ....) with same names (biology...) it makes it hard to opt out of archiving if they only see a list of same named courses. can we have a more detailed list?

2020 / Science / Biology / Microbiology 20201 / Science / Biology / Microbiology ....

Can you give a hint, where to look in the source code to try myself to add path to "%courses"

Syxton commented 3 months ago

@rsacher1 The function that returns the courselist in the emails is found here: https://github.com/Syxton/moodle-tool_coursearchiver/blob/master/classes/processor.php#L1350

Line 1387 uses $course->fullname so you would want to gather the category info first and then attach there.

rsacher1 commented 3 months ago

Thank you for your information! I managed to get the Paths into mails. And it works on my site as intended. I created a pull request

Am Mo., 12. Aug. 2024 um 17:36 Uhr schrieb Matt Davidson < @.***>:

@rsacher1 https://github.com/rsacher1 The function that returns the courselist in the emails is found here:

https://github.com/Syxton/moodle-tool_coursearchiver/blob/master/classes/processor.php#L1350

Line 1387 uses $course->fullname so you would want to gather the category info first and then attach there.

— Reply to this email directly, view it on GitHub https://github.com/Syxton/moodle-tool_coursearchiver/issues/61#issuecomment-2284305419, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADXXL3LKEL2FPGH3RVGARR3ZRDJBFAVCNFSM6AAAAABMAQ4X5KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBUGMYDKNBRHE . You are receiving this because you were mentioned.Message ID: @.***>

Syxton commented 2 months ago

@rsacher1 Glad I could help. FYI I don't see any pull request

rsacher1 commented 2 months ago

Ahhh... I created a fork... My mistake... Will look to create a branch instead... Not an experienced user

Matt Davidson @.***> schrieb am Mi., 4. Sep. 2024, 20:00:

@rsacher1 https://github.com/rsacher1 Glad I could help. FYI I don't see any pull request

— Reply to this email directly, view it on GitHub https://github.com/Syxton/moodle-tool_coursearchiver/issues/61#issuecomment-2329671058, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADXXL3PGYNO6WR3C6HDKZYLZU5DC7AVCNFSM6AAAAABMAQ4X5KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRZGY3TCMBVHA . You are receiving this because you were mentioned.Message ID: @.***>