Skip to content

Document user license requirements for @-mentions in SPE docs #10301

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

dluces
Copy link
Contributor

@dluces dluces commented Jun 13, 2025

Category

  • Content fix
  • New article

What's in this Pull Request?

Documents user license requirements for @mentions in Office documents for SPE.

Copy link
Contributor

Learn Build status updates of commit 8eb2b24:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/development/content-experiences/office-experience.md ✅Succeeded View

For more details, please refer to the build report.

Copy link
Contributor

Learn Build status updates of commit 97dc791:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/development/content-experiences/office-experience.md ✅Succeeded View

For more details, please refer to the build report.

> [!NOTE]
> Mentions require target users to [have an M365 license assigned to them](../auth.md#mention-users-in-office-documents).
>
> Mentions are restricted to people inside the consuming tenant's organization. Mentions excludes Guests and users from other tenants in a Multi-Tenant setting.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was copied directly from the @-mentions documentation, but having "Guests" (upper-case G) and "Multi-Tenant" (upper-case M and T) looks a bit off. I'll leave this up for reviewers, I'd change all of those to lower-case.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I suggest, not just for this, but for all content updates, is to run this through Acrolinx. You can learn more about this from the MSDocs/MSLearn MSFT-internal site.

In a nutshell, Acrolinx is like Grammarly, but for Microsoft's style guide. MSDocs uses it, and all content published to MSLearn's training section (modules, labs, etc), must have a score of 80+ to get published. It finds things like these.

In this case, it doesn't flag the uppercase "Guest", but the correct use of the other is "multitenant" (one word, all lowercase). I'll submit my changes from running this page through Acrolinx

@andrewconnell andrewconnell self-assigned this Jun 18, 2025
@andrewconnell andrewconnell added the pr:in-review Actively reviewing pull request label Jun 18, 2025
@andrewconnell andrewconnell self-requested a review June 18, 2025 13:14
Copy link
Contributor

Learn Build status updates of commit 6c6fb8b:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/development/content-experiences/office-experience.md ✅Succeeded View

For more details, please refer to the build report.

Copy link
Contributor

Learn Build status updates of commit 6dd3dd4:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/development/content-experiences/office-experience.md ✅Succeeded View

For more details, please refer to the build report.

@andrewconnell andrewconnell added pr:ready-to-merge Ready to merge PR and removed pr:in-review Actively reviewing pull request labels Jun 18, 2025
@andrewconnell andrewconnell merged commit 28b46f3 into SharePoint:main Jun 18, 2025
2 checks passed
@dluces dluces deleted the user/dluces/update_spe_m365_licence branch June 24, 2025 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:ready-to-merge Ready to merge PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants