Skip to content

Conversation

@dluces
Copy link
Contributor

@dluces dluces commented Jul 17, 2025

Category

  • Content fix
  • New article

What's in this Pull Request?

Update SPE docs to point to new Container Type Management APIs on MSGraph instead of powershell cmdlets.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 56452ff:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View

For more details, please refer to the build report.

@dluces dluces changed the title Update SPE auth page to incorporate new CT Mgmt APIs Update SPE auth page to incorporate new Container Type Management APIs Jul 17, 2025
@dluces dluces changed the title Update SPE auth page to incorporate new Container Type Management APIs Update SPE docs to incorporate new Container Type Management APIs Jul 17, 2025
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 20d5452:

💡 Validation status: suggestions

File Status Preview URL Details
docs/embedded/getting-started/containertypes.md 💡Suggestion View Details
docs/embedded/development/auth.md ✅Succeeded View

docs/embedded/getting-started/containertypes.md

  • Line 42, Column 200: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-post-containertypes' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 165, Column 62: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-post-containertypes' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 165, Column 183: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestoragecontainertype-update' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 170, Column 71: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/resources/filestoragecontainertypesettings' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 174, Column 113: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-list-containertypes' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 178, Column 245: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-post-containertyperegistrations' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 183, Column 95: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-delete-containertypes' points to a learn site page that doesn't exist. Check the path or URL and update the link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 96fb02d:

💡 Validation status: suggestions

File Status Preview URL Details
docs/embedded/getting-started/containertypes.md 💡Suggestion View Details
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/getting-started/register-api-documentation.md ✅Succeeded View

docs/embedded/getting-started/containertypes.md

  • Line 42, Column 200: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-post-containertypes' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 165, Column 62: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-post-containertypes' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 165, Column 183: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestoragecontainertype-update' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 170, Column 71: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/resources/filestoragecontainertypesettings' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 174, Column 113: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-list-containertypes' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 178, Column 245: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-post-containertyperegistrations' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 183, Column 95: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-delete-containertypes' points to a learn site page that doesn't exist. Check the path or URL and update the link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit bb54408:

💡 Validation status: suggestions

File Status Preview URL Details
docs/embedded/getting-started/containertypes.md 💡Suggestion View Details
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/getting-started/register-api-documentation.md ✅Succeeded View

docs/embedded/getting-started/containertypes.md

  • Line 42, Column 200: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-post-containertypes' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 165, Column 62: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-post-containertypes' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 165, Column 183: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestoragecontainertype-update' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 170, Column 71: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/resources/filestoragecontainertypesettings' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 174, Column 113: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-list-containertypes' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 178, Column 245: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-post-containertyperegistrations' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 183, Column 95: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/api/filestorage-delete-containertypes' points to a learn site page that doesn't exist. Check the path or URL and update the link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@dluces dluces force-pushed the user/dluces/use_ct_mgmt_apis branch from bb54408 to 446825c Compare August 11, 2025 16:03
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 446825c:

💡 Validation status: suggestions

File Status Preview URL Details
docs/embedded/getting-started/containertypes.md 💡Suggestion View Details
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/getting-started/register-api-documentation.md ✅Succeeded View

docs/embedded/getting-started/containertypes.md

  • Line 170, Column 71: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/resources/filestoragecontainertypesettings' points to a learn site page that doesn't exist. Check the path or URL and update the link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@dluces dluces force-pushed the user/dluces/use_ct_mgmt_apis branch from 0166259 to 5647201 Compare August 11, 2025 16:27
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 0166259:

💡 Validation status: suggestions

File Status Preview URL Details
docs/embedded/getting-started/containertypes.md 💡Suggestion View Details
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/getting-started/register-api-documentation.md ✅Succeeded View

docs/embedded/getting-started/containertypes.md

  • Line 170, Column 71: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/resources/filestoragecontainertypesettings' points to a learn site page that doesn't exist. Check the path or URL and update the link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 5647201:

💡 Validation status: suggestions

File Status Preview URL Details
docs/embedded/getting-started/containertypes.md 💡Suggestion View Details
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/getting-started/register-api-documentation.md ✅Succeeded View

docs/embedded/getting-started/containertypes.md

  • Line 170, Column 71: [Suggestion: learn-site-link-broken - See documentation] Link '/graph/resources/filestoragecontainertypesettings' points to a learn site page that doesn't exist. Check the path or URL and update the link.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 7b01459:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/getting-started/containertypes.md ✅Succeeded View
docs/embedded/getting-started/register-api-documentation.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 5953f03:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/embedded/administration/billing/billing.md ⚠️Warning View Details
docs/embedded/administration/developer-admin/dev-admin.md ⚠️Warning View Details
docs/embedded/getting-started/containertypes.md ⚠️Warning View Details
docs/embedded/administration/billing/meters.md ✅Succeeded View
docs/embedded/administration/consuming-tenant-admin/cta.md ✅Succeeded View
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/getting-started/register-api-documentation.md ✅Succeeded View
docs/embedded/getting-started/spembedded-for-vscode.md ✅Succeeded View

docs/embedded/administration/billing/billing.md

  • Line 39, Column 85: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#set-up-billing-for-passthrough-app' in 'embedded/administration/consuming-tenant-admin/cta.md'.

docs/embedded/administration/developer-admin/dev-admin.md

  • Line 45, Column 643: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#set-up-billing-for-passthrough-app' in 'embedded/administration/consuming-tenant-admin/cta.md'.

docs/embedded/getting-started/containertypes.md

  • Line 40, Column 4: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#standard-container-types-non-trial' in 'embedded/getting-started/containertypes.md', did you mean '#standard-container-types-nontrial'?

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

Choose a reason for hiding this comment

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

The title for the "Pay as you go message" doesn't seem related to the description, at least to me.

Choose a reason for hiding this comment

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

In the Egress section, it says that "to the data downloaded from the SharePoint Embedded platform to the customer's client device such as a desktop or mobile device.".

We have to be a bit more careful with this, to also include the case where an SPE service application makes calls that translate into Egress, but doesn't necessarily downloads that data to any end user device.

@dluces dluces force-pushed the user/dluces/use_ct_mgmt_apis branch from 5953f03 to bf5a68c Compare September 24, 2025 23:15
@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit bf5a68c:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/embedded/administration/billing/billing.md ⚠️Warning View Details
docs/embedded/administration/developer-admin/dev-admin.md ⚠️Warning View Details
docs/embedded/getting-started/containertypes.md ⚠️Warning View Details
docs/embedded/administration/billing/meters.md ✅Succeeded View
docs/embedded/administration/consuming-tenant-admin/cta.md ✅Succeeded View
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/getting-started/register-api-documentation.md ✅Succeeded View
docs/embedded/getting-started/spembedded-for-vscode.md ✅Succeeded View

docs/embedded/administration/billing/billing.md

  • Line 39, Column 85: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#set-up-billing-for-passthrough-app' in 'embedded/administration/consuming-tenant-admin/cta.md'.

docs/embedded/administration/developer-admin/dev-admin.md

  • Line 45, Column 643: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#set-up-billing-for-passthrough-app' in 'embedded/administration/consuming-tenant-admin/cta.md'.

docs/embedded/getting-started/containertypes.md

  • Line 40, Column 4: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#standard-container-types-non-trial' in 'embedded/getting-started/containertypes.md', did you mean '#standard-container-types-nontrial'?

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

Copy link
Contributor

@javieralvarezchiang javieralvarezchiang left a comment

Choose a reason for hiding this comment

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

I have yet to review containertypes.md in detail.

@andrewconnell
Copy link
Collaborator

Can we get a status on this PR? It's been in draft for months with no activity and now has a bunch of merge conflicts.

Please either close it if it no longer applies or let us know what's going on... otherwise we'll close it next week (where you can always reopen it in the future when you're ready for a review).

/cc @dluces @javieralvarezchiang @reynaldo-roel-ms

@dluces dluces force-pushed the user/dluces/use_ct_mgmt_apis branch from bf5a68c to 7cfb2eb Compare January 20, 2026 02:07
@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@dluces
Copy link
Contributor Author

dluces commented Jan 20, 2026

Thanks @andrewconnell ! I'm taking this on again this week. We were waiting for the APIs to go to v1.0 and now they are ;) Expect this to go out of draft in the next few days.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 7cfb2eb:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/embedded/administration/billing/billing.md ⚠️Warning View Details
docs/embedded/administration/developer-admin/dev-admin.md ⚠️Warning View Details
docs/embedded/development/tutorials/vendor-install-app-customer.md ⚠️Warning View Details
docs/embedded/getting-started/containertypes.md ⚠️Warning View Details
docs/embedded/administration/billing/meters.md ✅Succeeded View
docs/embedded/administration/consuming-tenant-admin/cta.md ✅Succeeded View
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/getting-started/register-api-documentation.md ✅Succeeded View
docs/embedded/getting-started/spembedded-for-vscode.md ✅Succeeded View

docs/embedded/administration/billing/billing.md

  • Line 39, Column 85: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#set-up-billing-for-passthrough-app' in 'embedded/administration/consuming-tenant-admin/cta.md'.

docs/embedded/administration/developer-admin/dev-admin.md

  • Line 45, Column 643: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#set-up-billing-for-passthrough-app' in 'embedded/administration/consuming-tenant-admin/cta.md'.

docs/embedded/development/tutorials/vendor-install-app-customer.md

  • Line 24, Column 43: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#set-up-billing-for-pass-through-app' in 'embedded/administration/consuming-tenant-admin/cta.md'.
  • Line 24, Column 207: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#pass-through' in 'embedded/administration/billing/billing.md'.
  • Line 62, Column 30: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#pass-through' in 'embedded/administration/billing/billing.md'.
  • Line 63, Column 65: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#set-up-billing-for-pass-through-app' in 'embedded/administration/consuming-tenant-admin/cta.md'.

docs/embedded/getting-started/containertypes.md

  • Line 40, Column 4: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#standard-container-types-non-trial' in 'embedded/getting-started/containertypes.md', did you mean '#standard-container-types-nontrial'?

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit e30f4a3:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/embedded/administration/billing/billing.md ⚠️Warning View Details
docs/embedded/administration/developer-admin/dev-admin.md ⚠️Warning View Details
docs/embedded/development/tutorials/vendor-install-app-customer.md ⚠️Warning View Details
docs/embedded/getting-started/containertypes.md ⚠️Warning View Details
docs/embedded/administration/billing/meters.md ✅Succeeded View
docs/embedded/administration/consuming-tenant-admin/cta.md ✅Succeeded View
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/getting-started/register-api-documentation.md ✅Succeeded View
docs/embedded/getting-started/spembedded-for-vscode.md ✅Succeeded View

docs/embedded/administration/billing/billing.md

  • Line 42, Column 85: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#set-up-billing-for-passthrough-app' in 'embedded/administration/consuming-tenant-admin/cta.md'.

docs/embedded/administration/developer-admin/dev-admin.md

  • Line 45, Column 643: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#set-up-billing-for-passthrough-app' in 'embedded/administration/consuming-tenant-admin/cta.md'.

docs/embedded/development/tutorials/vendor-install-app-customer.md

  • Line 24, Column 43: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#set-up-billing-for-pass-through-app' in 'embedded/administration/consuming-tenant-admin/cta.md'.
  • Line 24, Column 207: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#pass-through' in 'embedded/administration/billing/billing.md'.
  • Line 62, Column 30: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#pass-through' in 'embedded/administration/billing/billing.md'.
  • Line 63, Column 65: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#set-up-billing-for-pass-through-app' in 'embedded/administration/consuming-tenant-admin/cta.md'.

docs/embedded/getting-started/containertypes.md

  • Line 40, Column 4: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#standard-container-types-non-trial' in 'embedded/getting-started/containertypes.md', did you mean '#standard-container-types-nontrial'?

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@dluces dluces marked this pull request as ready for review January 20, 2026 02:35
@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit f7b944c:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/embedded/development/tutorials/vendor-install-app-customer.md ⚠️Warning View Details
docs/embedded/administration/billing/billing.md ✅Succeeded View
docs/embedded/administration/billing/meters.md ✅Succeeded View
docs/embedded/administration/consuming-tenant-admin/cta.md ✅Succeeded View
docs/embedded/administration/developer-admin/dev-admin.md ✅Succeeded View
docs/embedded/development/auth.md ✅Succeeded View
docs/embedded/getting-started/containertypes.md ✅Succeeded View
docs/embedded/getting-started/register-api-documentation.md ✅Succeeded View
docs/embedded/getting-started/spembedded-for-vscode.md ✅Succeeded View

docs/embedded/development/tutorials/vendor-install-app-customer.md

  • Line 24, Column 217: [Warning: bookmark-not-found - See documentation] Cannot find bookmark '#pass-through' in 'embedded/administration/billing/billing.md'.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

@Spucelik
Copy link
Contributor

Reviewing the PR, a few thoughts/corrections:

  1. On the /development/auth.md page, there is a reference to "register and application" but the link is a 404. I'm assuming this will still require you to modify the manifest file to add the billing profile in PowerShell?
  2. I would put a link to the app registration creation in the requirements section to avoid multiple clicks to find it.
  3. For the graph permission needed (FileStorageContainerType.Manage.All - Application) I think it's more readable if we have it in a table, like the Graph API reference
  4. When creating the container type, for {ApplicationId}, I would make it clear that this is the app id from the App Registration created earlier.
  5. In the Note section, last item ("SubscriptionNotRegistered error, it is because Microsoft.Syntex isn't registered")...provide a link the will guide them on how to configure this.
  6. Deleting container types - is this supported for standard container types?
  7. When registering a container type on a consuming tenant, 1st bullet of the Authentication and Authorization Requirements should contain a link on how to create a service principal.
  8. Once the Container Type is registered, there should be some guidance on creating permission grants

@dluces
Copy link
Contributor Author

dluces commented Jan 21, 2026

Thanks for the suggestions @Spucelik! I am unsure what parts you're commenting on, can you leave comments on the exact lines? Some notes though:

  1. The link for Register an application goes to "register an application with the Microsoft identity platform" page. I just tried it in the preview link for the auth page and it works fine. This part also has not changed in this PR.
  2. The "register an application" link is actually on the Pre-requisites section in the auth page. What page were you referring to here?
  3. Fair. The scope was not documented at all. I've added an "Application permissions" section linking to the documentation for the scopes. The table is nice, but I'd rather not duplicate the official documentation otherwise this will for sure get out of sync. The exception is the Container.Selected scope for which this page (auth.md) is the only official documentation.
  4. I have no idea what page/line this is referring to.
  5. They don't need to configure anything. So no need to do anything: "The cmdlet sends a resource provider registration request on your behalf but it takes a few minutes to be completed. Wait 5-10 minutes and try again until the cmdlet succeeds."
  6. Not yet :( Fixed page.
  7. Added link.
  8. What do you mean? The documented registration process comes with sample applicationPermissionGrants. Can you put this on a specific file/line to know where you'd see benefits from this?

Pushed udpates now

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

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.

5 participants