Skip to content

Conversation

@SteveMutungi254
Copy link
Contributor

Per user MFA validations:

  • Moved Get-EntraUserAuthenticationMethod to Signins folder.
  • Added tests.
  • Added mandatory parameter validations.
  • Added pre-checks for auth context.

@SteveMutungi254 SteveMutungi254 requested a review from a team as a code owner March 26, 2025 15:49
@learn-build-service-prod
Copy link

Learn Build status updates of commit 3e7d44a:

❌ Validation status: errors

Please follow instructions here which may help to resolve issue.

File Status Preview URL Details
❌Error Details

  • [Error: CannotMergeCommit] Cannot merge commit 3e7d44a536de734afc1bcd6e46c4b7ce97a02c26 in branch validations/per-user-mfa-commands of repository https://github.com/microsoftgraph/entra-powershell into branch main (commit 045d7fc5cc2b825f73f328bea430d0795b99e2f0). Please follow this documentation: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/ to use git.exe to resolve you content conflicts locally and then push to remote.

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.

For any questions, please:

@learn-build-service-prod
Copy link

Learn Build status updates of commit 7fa74af:

✅ Validation status: passed

File Status Preview URL Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-v1.0/SignIns/Get-EntraUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-preview)
module/Entra/Microsoft.Entra/DirectoryManagement/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded n/a (file deleted or renamed)
module/Entra/Microsoft.Entra/SignIns/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationMethod.ps1 ✅Succeeded
test/Entra/SignIns/Get-EntraUserAuthenticationMethod.Tests.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link

Learn Build status updates of commit b36dbae:

✅ Validation status: passed

File Status Preview URL Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-v1.0/SignIns/Get-EntraUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-preview)
module/Entra/Microsoft.Entra/DirectoryManagement/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded n/a (file deleted or renamed)
module/Entra/Microsoft.Entra/SignIns/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationMethod.ps1 ✅Succeeded
test/Entra/SignIns/Get-EntraUserAuthenticationMethod.Tests.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link

Learn Build status updates of commit 3e91d89:

✅ Validation status: passed

File Status Preview URL Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-v1.0/SignIns/Get-EntraUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-preview)
module/Entra/Microsoft.Entra/DirectoryManagement/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded n/a (file deleted or renamed)
module/Entra/Microsoft.Entra/SignIns/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationMethod.ps1 ✅Succeeded
test/Entra/SignIns/Get-EntraUserAuthenticationMethod.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Get-EntraBetaUserAuthenticationMethod.Tests.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link

Learn Build status updates of commit d097e71:

✅ Validation status: passed

File Status Preview URL Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-v1.0/SignIns/Get-EntraUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-preview)
module/Entra/Microsoft.Entra/DirectoryManagement/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded n/a (file deleted or renamed)
module/Entra/Microsoft.Entra/SignIns/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationMethod.ps1 ✅Succeeded
test/Entra/SignIns/Get-EntraUserAuthenticationMethod.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Get-EntraBetaUserAuthenticationMethod.Tests.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link

@learn-build-service-prod
Copy link

@learn-build-service-prod
Copy link

Learn Build status updates of commit f820bc5:

💡 Validation status: suggestions

File Status Preview URL Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.md 💡Suggestion View (entra-powershell-beta-preview) Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/SignIns/Update-EntraBetaUserAuthenticationRequirement.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-v1.0/SignIns/Get-EntraUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-preview)
module/Entra/Microsoft.Entra/DirectoryManagement/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded n/a (file deleted or renamed)
module/Entra/Microsoft.Entra/SignIns/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Update-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded
test/Entra/SignIns/Get-EntraUserAuthenticationMethod.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Get-EntraBetaUserAuthenticationMethod.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Get-EntraBetaUserAuthenticationRequirement.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Update-EntraBetaUserAuthenticationRequirement.Tests.ps1 ✅Succeeded

module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.md

  • Line 93, Column 319: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/entra/identity/authentication/tutorial-enable-azure-mfa' will be broken in isolated environments. Replace with a relative link.
  • Line 95, Column 73: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/entra/fundamentals/security-defaults' 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.

For any questions, please:

@learn-build-service-prod
Copy link

Learn Build status updates of commit d67c6fb:

💡 Validation status: suggestions

File Status Preview URL Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.md 💡Suggestion View (entra-powershell-beta-preview) Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/SignIns/Update-EntraBetaUserAuthenticationRequirement.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-v1.0/SignIns/Get-EntraUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-preview)
module/Entra/Microsoft.Entra/DirectoryManagement/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded n/a (file deleted or renamed)
module/Entra/Microsoft.Entra/SignIns/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Update-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded
test/Entra/SignIns/Get-EntraUserAuthenticationMethod.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Get-EntraBetaUserAuthenticationMethod.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Get-EntraBetaUserAuthenticationRequirement.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Update-EntraBetaUserAuthenticationRequirement.Tests.ps1 ✅Succeeded

module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.md

  • Line 93, Column 319: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/entra/identity/authentication/tutorial-enable-azure-mfa' will be broken in isolated environments. Replace with a relative link.
  • Line 95, Column 73: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/entra/fundamentals/security-defaults' 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.

For any questions, please:

@learn-build-service-prod
Copy link

Learn Build status updates of commit 0d5baf8:

💡 Validation status: suggestions

File Status Preview URL Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.md 💡Suggestion View (entra-powershell-beta-preview) Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/SignIns/Update-EntraBetaUserAuthenticationRequirement.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-v1.0/SignIns/Get-EntraUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-preview)
module/Entra/Microsoft.Entra/DirectoryManagement/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded n/a (file deleted or renamed)
module/Entra/Microsoft.Entra/SignIns/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Update-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded
test/Entra/SignIns/Get-EntraUserAuthenticationMethod.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Get-EntraBetaUserAuthenticationMethod.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Get-EntraBetaUserAuthenticationRequirement.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Update-EntraBetaUserAuthenticationRequirement.Tests.ps1 ✅Succeeded

module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.md

  • Line 93, Column 319: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/entra/identity/authentication/tutorial-enable-azure-mfa' will be broken in isolated environments. Replace with a relative link.
  • Line 95, Column 73: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/entra/fundamentals/security-defaults' 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.

For any questions, please:

@SteveMutungi254 SteveMutungi254 added Ready For Review The PR Ready for Review P0 Highest priority labels Mar 26, 2025
@learn-build-service-prod
Copy link

Learn Build status updates of commit 550522e:

💡 Validation status: suggestions

File Status Preview URL Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.md 💡Suggestion View (entra-powershell-beta-preview) Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/SignIns/Update-EntraBetaUserAuthenticationRequirement.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-v1.0/SignIns/Get-EntraUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-preview)
module/Entra/Microsoft.Entra/DirectoryManagement/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded n/a (file deleted or renamed)
module/Entra/Microsoft.Entra/SignIns/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Update-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded
test/Entra/SignIns/Get-EntraUserAuthenticationMethod.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Get-EntraBetaUserAuthenticationMethod.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Get-EntraBetaUserAuthenticationRequirement.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Update-EntraBetaUserAuthenticationRequirement.Tests.ps1 ✅Succeeded

module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.md

  • Line 93, Column 319: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/entra/identity/authentication/tutorial-enable-azure-mfa' will be broken in isolated environments. Replace with a relative link.
  • Line 95, Column 73: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/entra/fundamentals/security-defaults' 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.

For any questions, please:

@learn-build-service-prod
Copy link

Learn Build status updates of commit e7ae1bb:

💡 Validation status: suggestions

File Status Preview URL Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.md 💡Suggestion View (entra-powershell-beta-preview) Details
module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-beta/SignIns/Update-EntraBetaUserAuthenticationRequirement.md ✅Succeeded View (entra-powershell-beta-preview)
module/docs/entra-powershell-v1.0/SignIns/Get-EntraUserAuthenticationMethod.md ✅Succeeded View (entra-powershell-preview)
module/Entra/Microsoft.Entra/DirectoryManagement/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded n/a (file deleted or renamed)
module/Entra/Microsoft.Entra/SignIns/Get-EntraUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationMethod.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded
module/EntraBeta/Microsoft.Entra.Beta/SignIns/Update-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded
test/Entra/SignIns/Get-EntraUserAuthenticationMethod.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Get-EntraBetaUserAuthenticationMethod.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Get-EntraBetaUserAuthenticationRequirement.Tests.ps1 ✅Succeeded
test/EntraBeta/SignIns/Update-EntraBetaUserAuthenticationRequirement.Tests.ps1 ✅Succeeded

module/docs/entra-powershell-beta/SignIns/Get-EntraBetaUserAuthenticationRequirement.md

  • Line 93, Column 319: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/entra/identity/authentication/tutorial-enable-azure-mfa' will be broken in isolated environments. Replace with a relative link.
  • Line 95, Column 73: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/entra/fundamentals/security-defaults' 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.

For any questions, please:

@SteveMutungi254 SteveMutungi254 merged commit 4d067b3 into main Mar 28, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P0 Highest priority Ready For Review The PR Ready for Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants