Skip to content

Update provision-on-demand.md#2003

Open
precubcr wants to merge 1 commit into
MicrosoftDocs:mainfrom
precubcr:patch-1
Open

Update provision-on-demand.md#2003
precubcr wants to merge 1 commit into
MicrosoftDocs:mainfrom
precubcr:patch-1

Conversation

@precubcr

Copy link
Copy Markdown

2605290050001111

@prmerger-automator

Copy link
Copy Markdown
Contributor

@precubcr : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit ed4b7ef:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/identity-platform/tutorial-desktop-app-maui-sign-in-sign-out.md ⚠️Warning Details
docs/identity-platform/tutorial-mobile-app-maui-sign-in-sign-out.md ⚠️Warning Details
docs/identity/app-provisioning/provision-on-demand.md ⚠️Warning Details

docs/identity-platform/tutorial-desktop-app-maui-sign-in-sign-out.md

  • Line 50, Column 4: [Warning: invalid-code] The code snippet "~/../ms-identity-ciam-dotnet-tutorial/1-Authentication/2-sign-in-maui/Views/MainView.xaml" could not be found.
  • Line 69, Column 4: [Warning: invalid-code] The code snippet "~/../ms-identity-ciam-dotnet-tutorial/1-Authentication/2-sign-in-maui/Views/MainView.xaml.cs" could not be found.
  • Line 86, Column 4: [Warning: invalid-code] The code snippet "~/../ms-identity-ciam-dotnet-tutorial/1-Authentication/2-sign-in-maui/Views/ClaimsView.xaml" could not be found.
  • Line 100, Column 4: [Warning: invalid-code] The code snippet "~/../ms-identity-ciam-dotnet-tutorial/1-Authentication/2-sign-in-maui/Views/ClaimsView.xaml.cs" could not be found.

docs/identity-platform/tutorial-mobile-app-maui-sign-in-sign-out.md

  • Line 50, Column 4: [Warning: invalid-code] The code snippet "~/../ms-identity-ciam-dotnet-tutorial/1-Authentication/2-sign-in-maui/Views/MainView.xaml" could not be found.
  • Line 69, Column 4: [Warning: invalid-code] The code snippet "~/../ms-identity-ciam-dotnet-tutorial/1-Authentication/2-sign-in-maui/Views/MainView.xaml.cs" could not be found.
  • Line 86, Column 4: [Warning: invalid-code] The code snippet "~/../ms-identity-ciam-dotnet-tutorial/1-Authentication/2-sign-in-maui/Views/ClaimsView.xaml" could not be found.
  • Line 100, Column 4: [Warning: invalid-code] The code snippet "~/../ms-identity-ciam-dotnet-tutorial/1-Authentication/2-sign-in-maui/Views/ClaimsView.xaml.cs" could not be found.

docs/identity/app-provisioning/provision-on-demand.md

  • Line 23, Column 91: [Warning: file-not-found - See documentation] Invalid file link: '[url](https://learn.microsoft.com/en-us/entra/identity/enterprise-apps/overview-assign-app-owners)'.
  • Line 23, Column 91: [Suggestion: docs-link-absolute - See documentation] Absolute link '[url](https://learn.microsoft.com/en-us/entra/identity/enterprise-apps/overview-assign-app-owners)' will be broken in isolated environments. Replace with a relative 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.



1. Sign in to the [Microsoft Entra admin center](https://entra.microsoft.com) as at least an [Application Administrator](~/identity/role-based-access-control/permissions-reference.md#application-administrator).
1. Sign in to the [Microsoft Entra admin center](https://entra.microsoft.com) as at least [Application Owner]([url](https://learn.microsoft.com/en-us/entra/identity/enterprise-apps/overview-assign-app-owners)) or an [Application Administrator](~/identity/role-based-access-control/permissions-reference.md#application-administrator).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
1. Sign in to the [Microsoft Entra admin center](https://entra.microsoft.com) as at least [Application Owner]([url](https://learn.microsoft.com/en-us/entra/identity/enterprise-apps/overview-assign-app-owners)) or an [Application Administrator](~/identity/role-based-access-control/permissions-reference.md#application-administrator).
1. Sign in to the [Microsoft Entra admin center](https://entra.microsoft.com) as at least [Application Owner]((/entra/identity/enterprise-apps/overview-assign-app-owners) or an [Application Administrator](/identity/role-based-access-control/permissions-reference.md#application-administrator).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change the Application Owner url to:
url

@v-regandowner

Copy link
Copy Markdown
Contributor

@jenniferf-skc

Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team



1. Sign in to the [Microsoft Entra admin center](https://entra.microsoft.com) as at least an [Application Administrator](~/identity/role-based-access-control/permissions-reference.md#application-administrator).
1. Sign in to the [Microsoft Entra admin center](https://entra.microsoft.com) as at least [Application Owner]([url](https://learn.microsoft.com/en-us/entra/identity/enterprise-apps/overview-assign-app-owners)) or an [Application Administrator](~/identity/role-based-access-control/permissions-reference.md#application-administrator).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change the Application Owner url to:
url

@jenniferf-skc

Copy link
Copy Markdown
Contributor

@jenniferf-skc

Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team

I've made a recommendation for updating the URL in the App Provisioning article. For the tutorials, I suggest #assign:@Henrymbugua review those code snippet warnings.

@prmerger-automator

Copy link
Copy Markdown
Contributor

The following users don't have valid GitHub IDs, or are not collaborators on this repo: henrymbugua review those code snippet warnings.

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.

4 participants