Skip to content

adds changes missed in #915#917

Closed
dstrome wants to merge 2 commits intomainfrom
dstrome-article-fixes
Closed

adds changes missed in #915#917
dstrome wants to merge 2 commits intomainfrom
dstrome-article-fixes

Conversation

@dstrome
Copy link
Copy Markdown
Contributor

@dstrome dstrome commented Jun 17, 2025

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit a8f74c9:

✅ Validation status: passed

File Status Preview URL Details
sharepoint/sharepoint-ps/sharepoint-online/Connect-SPOService.md ✅Succeeded View (sharepoint-ps)

For more details, please refer to the build report.

@dstrome
Copy link
Copy Markdown
Contributor Author

dstrome commented Jun 17, 2025

@samkabue - made the outstanding changes. Used "a" instead of "the" SharePoint administrator because "the" implies that there's only a single administrator. Also, checking with https://learn.microsoft.com/en-us/sharepoint/dev/embedded/administration/adminrole, "administrator" shouldn't be capitalized so I've made that change as well.

Other than those, I've added the embedded administrator role where there were "SharePoint administrator" references, updated common parameters, and switched the absolute learn links to relative.

@samkabue
Copy link
Copy Markdown
Contributor

Thanks @dstrome!

re: role names, I think we use the official Entra names, which have capitalization -- see https://learn.microsoft.com/en-us/entra/identity/role-based-access-control/permissions-reference

Also, the official name of SPO admin is "SharePoint Administrator" -- a bunch of places still use "SharePoint Online administrator" -- I think the "online" part was removed from the official Entra role name

@dstrome
Copy link
Copy Markdown
Contributor Author

dstrome commented Jun 17, 2025

@samkabue - I'll defer to your judgement on the correct capitalization and role names as SharePoint content is outside my scope.

@samkabue
Copy link
Copy Markdown
Contributor

if we could change all instances of "SharePoint Online administrator" to "SharePoint Administrator" that'd match our usage and documentation.

similarly, "SharePoint Embedded administrator" -> "SharePoint Embedded Administrator"

Thanks!

@samkabue
Copy link
Copy Markdown
Contributor

@dstrome @ShreyasSar26 will this be completed soon?

@samkabue
Copy link
Copy Markdown
Contributor

@dstrome this can be closed now, since @ShreyasSar26 made the changes in #918

@samkabue samkabue closed this Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants