Skip to content

Add new patch and update rules to match tutor style in extra LMSs#185

Merged
MoisesGSalas merged 5 commits into
mainfrom
dmh/refactor-extra-lms-rules
May 26, 2026
Merged

Add new patch and update rules to match tutor style in extra LMSs#185
MoisesGSalas merged 5 commits into
mainfrom
dmh/refactor-extra-lms-rules

Conversation

@DeimerM
Copy link
Copy Markdown
Contributor

@DeimerM DeimerM commented May 25, 2026

This PR ensures that Caddyfile rules for extra LMSs follow the same structure as standard Tutor rules. This guarantees that rules applied via inline plugins are consistently enforced across all sites on the platform.

Tutor style: https://github.com/overhangio/tutor/blob/f9d5f5b6dbada709368b496525d9232c3be6bc49/tutor/templates/apps/caddy/Caddyfile#L57-L65

@DeimerM DeimerM requested a review from a team as a code owner May 25, 2026 21:01
Copy link
Copy Markdown
Member

@MoisesGSalas MoisesGSalas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add changelog entry describing the changes you are making. We don't have contributing guidelines up to date at the moment, but you can use this as a reference: https://github.com/eduNEXT/tutor-contrib-codejail/blob/main/CONTRIBUTING.md#commit-and-pull-request-guidelines

Comment thread drydock/patches/caddyfile Outdated
@DeimerM
Copy link
Copy Markdown
Contributor Author

DeimerM commented May 25, 2026

Please add changelog entry describing the changes you are making. We don't have contributing guidelines up to date at the moment, but you can use this as a reference: https://github.com/eduNEXT/tutor-contrib-codejail/blob/main/CONTRIBUTING.md#commit-and-pull-request-guidelines

Thanks a lot for your review, I have added the changelog entry.

Comment thread changelog.d/20260525_165246_deimer.morales_refactor_extra_lms_rules.md Outdated
Comment thread changelog.d/20260525_165246_deimer.morales_refactor_extra_lms_rules.md Outdated
DeimerM and others added 2 commits May 26, 2026 11:26
…_lms_rules.md

Co-authored-by: Moisés González <moises.gonzalez@edunext.co>
Co-authored-by: Moisés González <moises.gonzalez@edunext.co>
@MoisesGSalas MoisesGSalas merged commit a46cdd5 into main May 26, 2026
10 checks passed
@MoisesGSalas MoisesGSalas deleted the dmh/refactor-extra-lms-rules branch May 26, 2026 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants