Skip to content

Mdl 87023 main#1

Closed
crazyserver wants to merge 2 commits into
mainfrom
MDL-87023-main
Closed

Mdl 87023 main#1
crazyserver wants to merge 2 commits into
mainfrom
MDL-87023-main

Conversation

@crazyserver

Copy link
Copy Markdown
Owner

*** PLEASE DO NOT OPEN PULL REQUESTS VIA GITHUB ***

The moodle.git repository at Github is just a mirror of the official repository. We do not accept pull requests at Github.

See CONTRIBUTING.md guidelines for how to contribute patches for Moodle. Thank you.

--

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

Updates the tool_mobile plugin to align feature identifiers and UI wording with the newer “site plugins” terminology and delegate naming used by the Moodle app, including an upgrade step to migrate existing disabledfeatures configuration.

Changes:

  • Bump tool_mobile plugin version to 2026031000.
  • Rename the “Remote add-ons” feature group label to “Site plugins”, while keeping the old string key as deprecated.
  • Add an upgrade step to migrate tool_mobile/disabledfeatures values to the new delegate/feature identifiers, and update the API feature list accordingly.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
public/admin/tool/mobile/version.php Increments plugin version to trigger the upgrade step.
public/admin/tool/mobile/lang/en/tool_mobile.php Introduces siteplugins string and keeps remoteaddons as deprecated.
public/admin/tool/mobile/db/upgrade.php Adds migration logic for existing disabledfeatures config values.
public/admin/tool/mobile/classes/api.php Updates feature keys/labels and modernizes some array initializations.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread public/admin/tool/mobile/classes/api.php Outdated
Comment thread public/admin/tool/mobile/db/upgrade.php
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