Skip to content

Conversation

arrivant
Copy link

@arrivant arrivant commented Sep 1, 2025

Customers are complaining that they are getting lost in this guideline, so this change aims to bring some clarity around temporary mocked values handling and SAML Signing certificate.

Customers are complaining that they are getting lost in two steps, so this change aims to bring some clarity around temporary mocked values handling and SAML Signing certificate
Copy link
Contributor

Learn Build status updates of commit 7cacebb:

💡 Validation status: suggestions

File Status Preview URL Details
docs/identity/saas-apps/alexishr-tutorial.md 💡Suggestion Details

docs/identity/saas-apps/alexishr-tutorial.md

  • Line 92, Column 4: [Suggestion: table-syntax-invalid - See documentation] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines.

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.

Updated the required attribute for Name ID format in the tutorial.
@arrivant
Copy link
Author

arrivant commented Sep 1, 2025

@microsoft-github-policy-service agree company="Simployer"

Copy link
Contributor

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

Copy link
Contributor

Learn Build status updates of commit 2e32f20:

✅ Validation status: passed

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

For more details, please refer to the build report.

Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

Copy link
Contributor

Learn Build status updates of commit a564f91:

✅ Validation status: passed

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

For more details, please refer to the build report.

Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

Updated the steps for configuring Microsoft Entra SSO with AlexisHR, including changes to the numbering and clarifications on placeholder values.
Copy link
Contributor

Learn Build status updates of commit e5967ff:

✅ Validation status: passed

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

For more details, please refer to the build report.

Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@v-regandowner
Copy link
Contributor

@nguhiu - 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

Copy link
Contributor

Learn Build status updates of commit 23c25e3:

✅ Validation status: passed

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

For more details, please refer to the build report.

Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@arrivant
Copy link
Author

@nguhiu @v-regandowner Could I ask you to review those changes? Customers are quite persistent in making mistakes when going through the current version of guidance, and we believe those few tweaks that are made in this PR should decrease the number of issues.

@arrivant
Copy link
Author

#sign-off

Copy link
Contributor

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

@ttorble
Copy link
Contributor

ttorble commented Sep 29, 2025

@nguhiu @v-regandowner Could I ask you to review those changes? Customers are quite persistent in making mistakes when going through the current version of guidance, and we believe those few tweaks that are made in this PR should decrease the number of issues.

@nguhiu can you respond the the comment above?

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves the clarity of the AlexisHR SSO tutorial documentation by addressing customer confusion around temporary mock values and SAML certificate handling. The changes restructure the configuration process to make it clearer that users need to start with placeholder values, configure AlexisHR, and then return to update Microsoft Entra ID with real values.

Key changes:

  • Restructured the SSO configuration workflow to clearly separate initial mock setup from final configuration with real values
  • Added explicit guidance on handling temporary placeholder values and when to replace them
  • Improved clarity around SAML certificate requirements and formatting

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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