Add options to show links to additional Microsoft apps in the Microsoft block - #3373
Open
Lai Wei (weilai-irl) wants to merge 1 commit into
Open
Add options to show links to additional Microsoft apps in the Microsoft block#3373Lai Wei (weilai-irl) wants to merge 1 commit into
Lai Wei (weilai-irl) wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Pull request overview
This PR expands the existing Microsoft block by adding optional (hidden-by-default) links to additional Microsoft 365 and Viva applications, including tenant-configurable URLs for Viva Amplify, Viva Connections, and Viva Learning.
Changes:
- Adds new admin settings (checkboxes + required URL fields where applicable) and new language strings for the additional app links.
- Extends the block rendering to conditionally show the new app links when enabled, and adds corresponding CSS/icon assets.
- Introduces regex validation constants for tenant-configured Viva URLs.
Reviewed changes
Copilot reviewed 7 out of 19 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| blocks/microsoft/version.php | Updates plugin version (currently uses 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 additional app links and required URL fields. |
| 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. |
| blocks/microsoft/lib.php | Adds regex constants for validating tenant-configured Viva URLs. |
| blocks/microsoft/lang/en/block_microsoft.php | Adds strings for new settings and new link labels, plus validation error text. |
| blocks/microsoft/classes/admin_setting_required_url.php | Adds a custom admin setting type for “required when enabled” URL validation. |
| blocks/microsoft/block_microsoft.php | Adds conditional rendering of the new app links in the connected-state block content. |
Suppressed comments (3)
blocks/microsoft/block_microsoft.php:448
- Links opened with target="_blank" should also set rel="noopener noreferrer" to prevent reverse-tabnabbing and avoid leaking window.opener to the external site.
$vivaamplifyattrs = ['target' => '_blank', 'class' => 'servicelink block_microsoft_vivaamplify'];
blocks/microsoft/block_microsoft.php:461
- Links opened with target="_blank" should also set rel="noopener noreferrer" to prevent reverse-tabnabbing and avoid leaking window.opener to the external site.
$vivaconnectionsattrs = ['target' => '_blank', 'class' => 'servicelink block_microsoft_vivaconnections'];
blocks/microsoft/block_microsoft.php:474
- Links opened with target="_blank" should also set rel="noopener noreferrer" to prevent reverse-tabnabbing and avoid leaking window.opener to the external site.
$vivalearningattrs = ['target' => '_blank', 'class' => 'servicelink block_microsoft_vivalearning'];
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Lai Wei (weilai-irl)
force-pushed
the
wip-135731-m500
branch
from
August 7, 2026 13:43
78e0cad to
5d4b9c5
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.