Skip to content
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

Hotfix - Update-EntraBetaUserAuthenticationRequirement #1251

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

SteveMutungi254
Copy link
Collaborator

@SteveMutungi254 SteveMutungi254 commented Dec 11, 2024

  • Correcting wrong parameter check (CurrentPassword instead of PerUserMfaState).
  • Removed validateset options to cater for future values.

@SteveMutungi254 SteveMutungi254 requested a review from a team as a code owner December 11, 2024 08:17
Copy link

Learn Build status updates of commit 19b49f3:

✅ Validation status: passed

File Status Preview URL Details
module/EntraBeta/AdditionalFunctions/Update-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@SteveMutungi254 SteveMutungi254 added Ready For Review The PR Ready for Review P0 Highest priority labels Dec 11, 2024
Copy link

Learn Build status updates of commit f5057ae:

✅ Validation status: passed

File Status Preview URL Details
module/EntraBeta/AdditionalFunctions/Update-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link

Learn Build status updates of commit 97dc57f:

✅ Validation status: passed

File Status Preview URL Details
module/EntraBeta/AdditionalFunctions/Update-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link

Learn Build status updates of commit 4cac54e:

✅ Validation status: passed

File Status Preview URL Details
module/EntraBeta/AdditionalFunctions/Update-EntraBetaUserAuthenticationRequirement.ps1 ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

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.

3 participants