Skip to content

Conversation

ChristianCB83
Copy link
Contributor

We must include the capability of using Authentication context for role elevation via PIM as the current version doesn't specify this anywhere. I had the chance to work on a case where customer was lost about this topic, since the doc was not covering it.

We must include the capability of using Authentication context for PIM as the current version doesn't specify this anywhere. I had the chance to work on a case where customer was lost about this topic, since the doc was not covering it.
Copy link
Contributor

@ChristianCB83 : 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 653bc31:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/identity/conditional-access/concept-conditional-access-cloud-apps.md ⚠️Warning Details

docs/identity/conditional-access/concept-conditional-access-cloud-apps.md

  • Line 241, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'id-governance/privileged-identity-management/pim-resource-roles-configure-role-settings'.

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.

Attempt to fix issue reported on the link, by removing the extension ".md"
Copy link
Contributor

Learn Build status updates of commit 6555ac0:

⚠️ Validation status: warnings

File Status Preview URL Details
docs/identity/conditional-access/concept-conditional-access-cloud-apps.md ⚠️Warning Details

docs/identity/conditional-access/concept-conditional-access-cloud-apps.md

  • Line 241, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'id-governance/privileged-identity-management/pim-resource-roles-configure-role-settings'.

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

@MicrosoftGuyJFlo MicrosoftGuyJFlo left a comment

Choose a reason for hiding this comment

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

Need to fix the URL for PIM. Then approved.

Copy link
Contributor

Learn Build status updates of commit 8b09419:

✅ Validation status: passed

File Status Preview URL Details
docs/identity/conditional-access/concept-conditional-access-cloud-apps.md ✅Succeeded

For more details, please refer to the build report.

- [Use sensitivity labels to protect content in Microsoft Teams, Microsoft 365 groups, and SharePoint sites](/purview/sensitivity-labels-teams-groups-sites)
- [Microsoft Defender for Cloud Apps](/defender-cloud-apps/session-policy-aad?branch=pr-en-us-2082#require-step-up-authentication-authentication-context)
- [Custom applications](~/identity-platform/developer-guide-conditional-access-authentication-context.md)
- [Priviledged Identity Management - On activation, require Microsoft Entra Conditional Access authentication context](/entra/id-governance/privileged-identity-management/pim-resource-roles-configure-role-settings#on-activation-require-microsoft-entra-conditional-access-authentication-context)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- [Priviledged Identity Management - On activation, require Microsoft Entra Conditional Access authentication context](/entra/id-governance/privileged-identity-management/pim-resource-roles-configure-role-settings#on-activation-require-microsoft-entra-conditional-access-authentication-context)
- [Privileged Identity Management - On activation, require Microsoft Entra Conditional Access authentication context](/entra/id-governance/privileged-identity-management/pim-resource-roles-configure-role-settings#on-activation-require-microsoft-entra-conditional-access-authentication-context)

fixing typo

@v-dirichards
Copy link
Contributor

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

@ChristianCB83
Copy link
Contributor Author

@lisaychuang , is there anything missing before we commit this change?

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.

3 participants