Skip to content

Update alvao-provisioning-tutorial.md #1502

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
merged 1 commit into from
May 5, 2025

Conversation

dominikforal
Copy link
Contributor

Updated links to user guide for current version and few other minor changes.

Copy link
Contributor

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

Copy link
Contributor

Learn Build status updates of commit 8d49384:

✅ Validation status: passed

File Status Preview URL Details
docs/identity/saas-apps/alvao-provisioning-tutorial.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@ttorble
Copy link
Contributor

ttorble commented Apr 15, 2025

@owinoakelo

Can you review the proposed changes? Note that the author metadata should reflect your GitHub alias. Can you create a PR in the private repo to update this? Thanks.

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

@owinoakelo
Copy link
Contributor

@ZollnerdMSFT Please review

@v-dirichards
Copy link
Contributor

@ZollnerdMSFT Please review

@ZollnerdMSFT - Could you review this proposed update to the article and enter #sign-off in a comment if it's ready to merge?

@ZollnerdMSFT
Copy link
Contributor

#sign-off

Copy link
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @thomasakelo

@@ -91,7 +91,7 @@ This section guides you through the steps to configure the Microsoft Entra provi

1. Select **Save**.

1. Under the **Mappings** section, select **Synchronize Microsoft Entra users to ALVAO**.
1. Under the **Mappings** section, select **Provision Azure Active Directory Users**.
Copy link
Contributor

@v-dirichards v-dirichards Apr 30, 2025

Choose a reason for hiding this comment

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

@ZollnerdMSFT Is this change valid? This change seems to be a content regression -- Azure Active Directory is a deprecated term. #hold-off

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@v-dirichards @ZollnerdMSFT It is deprecated term, but i can still see it in Provision settings in Azure portal:
obrazek

I can revert it back, but in any case this should be fixed on the Azure portal. Term "Synchronize Microsoft Entra users to ALVAO" is not present on the page so the guide is not accurate on this.


1. Under the **Mappings** section, select **Synchronize Microsoft Entra groups to ALVAO**.
1. Under the **Mappings** section, select **Provision Azure Active Directory Groups**.
Copy link
Contributor

Choose a reason for hiding this comment

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

@ZollnerdMSFT Is this change valid? This change seems to be a content regression -- Azure Active Directory is a deprecated term.

Copy link
Contributor

You typed hold-off, but the PR wasn't signed off, so nothing was changed.

@v-dirichards v-dirichards merged commit a3d173c into MicrosoftDocs:main May 5, 2025
2 checks passed
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.

5 participants