Skip to content

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

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

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

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:32
@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

Adds configurable options to surface additional Microsoft 365 + Viva application links (with new icons) in the Microsoft block, including support for tenant-specific URLs for certain Viva apps.

Changes:

  • Added new block settings to toggle additional Microsoft 365/Viva links, including required URL settings for Viva Amplify/Connections/Learning.
  • Rendered the new links in the connected-user block view and added CSS/icon assets for each app.
  • Introduced regex validation constants and a custom admin setting type to require URLs only when the companion “Show” option 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 Bumps plugin version for the new feature set.
blocks/microsoft/styles.css Adds CSS classes to display icons for the new app links.
blocks/microsoft/settings.php Adds admin settings to enable/disable new links and configure required tenant-specific URLs.
blocks/microsoft/lib.php Defines regex constants for validating tenant-configured Viva URLs.
blocks/microsoft/lang/en/block_microsoft.php Adds language strings for new links, settings, and validation error message.
blocks/microsoft/classes/admin_setting_required_url.php Adds a custom admin setting that requires a URL only when a companion checkbox is enabled.
blocks/microsoft/block_microsoft.php Adds the new Microsoft 365/Viva links to the connected-user block content.
blocks/microsoft/pix/word.svg Adds Word icon.
blocks/microsoft/pix/excel.svg Adds Excel icon.
blocks/microsoft/pix/powerpoint.svg Adds PowerPoint icon.
blocks/microsoft/pix/vivaengage.svg Adds Viva Engage icon.
blocks/microsoft/pix/vivaamplify.svg Adds Viva Amplify icon.
blocks/microsoft/pix/vivaconnections.svg Adds Viva Connections icon.
blocks/microsoft/pix/vivaglint.svg Adds Viva Glint icon.
blocks/microsoft/pix/vivagoals.svg Adds Viva Goals icon.
blocks/microsoft/pix/vivainsights.svg Adds Viva Insights icon.
blocks/microsoft/pix/vivalearning.svg Adds Viva Learning icon.
blocks/microsoft/pix/vivapulse.svg Adds Viva Pulse icon.
blocks/microsoft/pix/linkedin.svg Adds LinkedIn icon.
Suppressed comments (3)

blocks/microsoft/block_microsoft.php:448

  • This link opens in a new tab (target="_blank") but does not set rel="noopener noreferrer", which leaves users vulnerable to reverse-tabnabbing from the opened site. Add the rel attribute for external links.
                $vivaamplifyattrs = ['target' => '_blank', 'class' => 'servicelink block_microsoft_vivaamplify'];

blocks/microsoft/block_microsoft.php:461

  • This link opens in a new tab (target="_blank") but does not set rel="noopener noreferrer", which leaves users vulnerable to reverse-tabnabbing from the opened site. Add the rel attribute for external links.
                $vivaconnectionsattrs = ['target' => '_blank', 'class' => 'servicelink block_microsoft_vivaconnections'];

blocks/microsoft/block_microsoft.php:474

  • This link opens in a new tab (target="_blank") but does not set rel="noopener noreferrer", which leaves users vulnerable to reverse-tabnabbing from the opened site. Add the rel attribute for external links.
                $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/block_microsoft.php Outdated
Comment thread blocks/microsoft/version.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.

New icons and applications.

2 participants