Skip to content

issue#14 exclude module types from the back up - #20

Merged
dmitriim merged 3 commits into
mainfrom
issue#14_main
Aug 13, 2026
Merged

issue#14 exclude module types from the back up#20
dmitriim merged 3 commits into
mainfrom
issue#14_main

Conversation

@tuanngocnguyen

Copy link
Copy Markdown

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds per-course “excluded_mods” support so specific activity module types can be omitted from generated course backups, wiring the value from CSV upload through storage, backup execution, UI display, and privacy export.

Changes:

  • Add excluded_mods column to the plugin DB schema (install + upgrade) and model property.
  • Parse optional excluded_mods from uploaded CSV rows and display it in the migration table.
  • Apply module-level exclusions during backup plan execution and extend automated tests/docs accordingly.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
version.php Bumps plugin version for the schema/code change.
db/upgrade.php Adds excluded_mods field on upgrade savepoint.
db/install.xml Adds excluded_mods field to fresh installs (but VERSION needs correction).
classes/coursemigration.php Registers excluded_mods as a nullable text property.
classes/upload_course_list.php Supports optional excluded_mods column in CSV processing.
classes/task/course_backup.php Skips specified activity types when building the backup plan.
classes/output/coursemigration_table.php Adds excluded_mods column to admin table output.
classes/privacy/provider.php Includes excluded_mods in exported user data metadata and export.
lang/en/tool_coursemigration.php Adds UI/help/privacy strings for excluded_mods.
README.md Documents the new excluded_mods CSV feature (minor grammar issue).
example.csv Updates sample CSV to include excluded_mods.
tests/upload_course_list_test.php Extends CSV upload tests to validate excluded_mods persistence.
tests/task/course_backup_test.php Adds backup-content assertions to verify excluded modules aren’t backed up (one test name typo).
tests/privacy/provider_test.php Updates privacy export test coverage for the new field.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread classes/output/coursemigration_table.php
Comment thread README.md Outdated
Comment thread tests/task/course_backup_test.php Outdated
@tuanngocnguyen
tuanngocnguyen marked this pull request as draft August 10, 2026 06:44
@tuanngocnguyen
tuanngocnguyen removed the request for review from dmitriim August 10, 2026 06:44
@tuanngocnguyen tuanngocnguyen self-assigned this Aug 10, 2026
@tuanngocnguyen
tuanngocnguyen marked this pull request as ready for review August 10, 2026 06:57
@tuanngocnguyen

Copy link
Copy Markdown
Author

Hi @dmitriim ,

Would you please review once you got time?

@dmitriim

Copy link
Copy Markdown
Member

@tuanngocnguyen can you please have a look at failing CI?

@tuanngocnguyen
tuanngocnguyen force-pushed the issue#14_main branch 17 times, most recently from fddf075 to c0395fa Compare August 13, 2026 01:56
@tuanngocnguyen
tuanngocnguyen force-pushed the issue#14_main branch 2 times, most recently from b38a1f6 to e805979 Compare August 13, 2026 02:41
@dmitriim
dmitriim merged commit f681f2e into main Aug 13, 2026
31 checks passed
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.

4 participants