Skip to content

Add options to show links to additional Microsoft apps in the Microsoft block - #3372

Open
Lai Wei (weilai-irl) wants to merge 1 commit into
MOODLE_405_STABLEfrom
wip-135731-m405
Open

Add options to show links to additional Microsoft apps in the Microsoft block#3372
Lai Wei (weilai-irl) wants to merge 1 commit into
MOODLE_405_STABLEfrom
wip-135731-m405

Conversation

@weilai-irl

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI lite review requested due to automatic review settings August 7, 2026 13:30
@weilai-irl Lai Wei (weilai-irl) self-assigned this Aug 7, 2026
@weilai-irl Lai Wei (weilai-irl) added this to the 2026-03 milestone Aug 7, 2026
@weilai-irl Lai Wei (weilai-irl) linked an issue Aug 7, 2026 that may be closed by this pull request

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

This PR extends the Moodle block_microsoft plugin to optionally show links to additional Microsoft 365 and Viva apps within the Microsoft block, controlled by new admin settings (including tenant-specific URL settings for some Viva apps).

Changes:

  • Adds admin settings (checkboxes) to enable/disable additional Microsoft 365/Viva links, plus required URL settings for Viva Amplify/Connections/Learning.
  • Renders the new links in the block when enabled, and adds corresponding CSS + SVG icons.
  • Introduces URL-validation regex constants and a custom admin setting type to require a URL when its companion “Show …” checkbox is enabled.

Reviewed changes

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

Show a summary per file
File Description
blocks/microsoft/version.php Updates plugin version (currently to a decimal value).
blocks/microsoft/styles.css Adds icon CSS classes for the new app links.
blocks/microsoft/settings.php Adds new admin settings for showing app links + configuring tenant URLs.
blocks/microsoft/lib.php Adds regex constants used to validate tenant-specific Viva URLs.
blocks/microsoft/lang/en/block_microsoft.php Adds new UI strings for the new links/settings and URL-required error.
blocks/microsoft/classes/admin_setting_required_url.php Adds a custom admin setting to require a URL when a related checkbox is enabled.
blocks/microsoft/block_microsoft.php Adds the new app links to the block output when enabled.
blocks/microsoft/pix/*.svg Adds new SVG icons for the additional apps.
Suppressed comments (3)

blocks/microsoft/block_microsoft.php:448

  • Links opened with target="_blank" should include rel="noopener noreferrer" to prevent reverse-tabnabbing.
                $vivaamplifyattrs = ['target' => '_blank', 'class' => 'servicelink block_microsoft_vivaamplify'];

blocks/microsoft/block_microsoft.php:461

  • Links opened with target="_blank" should include rel="noopener noreferrer" to prevent reverse-tabnabbing.
                $vivaconnectionsattrs = ['target' => '_blank', 'class' => 'servicelink block_microsoft_vivaconnections'];

blocks/microsoft/block_microsoft.php:474

  • Links opened with target="_blank" should include rel="noopener noreferrer" to prevent reverse-tabnabbing.
                $vivalearningattrs = ['target' => '_blank', 'class' => 'servicelink block_microsoft_vivalearning'];

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

Comment thread blocks/microsoft/version.php
Comment thread blocks/microsoft/block_microsoft.php Outdated
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.

New icons and applications.

2 participants