Skip to content

Bugfix-1082 #1091

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Sep 24, 2024
Merged

Bugfix-1082 #1091

merged 5 commits into from
Sep 24, 2024

Conversation

@v-akarke
Copy link
Collaborator

@v-akarke v-akarke commented Sep 18, 2024

No description provided.

@v-akarke v-akarke requested a review from a team as a code owner September 18, 2024 05:49
@learn-build-service-prod
Copy link

learn-build-service-prod bot commented Sep 18, 2024

Learn Build status updates of commit 3933f9e:

✅ Validation status: passed

File Status Preview URL Details
module/Entra/AdditionalFunctions/Find-EntraPermission.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAdministrativeUnit.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAdministrativeUnitMember.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraApplicationTemplate.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAuditDirectoryLog.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAuditSignInLog.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraObjectSetting.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraApplication.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraApplicationOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraContact.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeletedApplication.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeviceRegisteredOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeviceRegisteredUser.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDirectoryRoleDefinition.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraGroupMember.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraGroupOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraServicePrincipalOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraTenantDetail.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUser.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserCreatedObject.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserDirectReport.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserOwnedDevice.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserOwnedObject.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserRegisteredDevice.ps1 ✅Succeeded
module/EntraBeta/AdditionalFunctions/Get-EntraBetaApplicationProxyConnector.ps1 ✅Succeeded

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link

learn-build-service-prod bot commented Sep 18, 2024

Learn Build status updates of commit 4c43544:

✅ Validation status: passed

File Status Preview URL Details
module/Entra/AdditionalFunctions/Find-EntraPermission.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAdministrativeUnit.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAdministrativeUnitMember.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraApplicationTemplate.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAuditDirectoryLog.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAuditSignInLog.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraObjectSetting.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraPolicy.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraApplication.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraApplicationOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraContact.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeletedApplication.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeviceRegisteredOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeviceRegisteredUser.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDirectoryRoleDefinition.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraGroupMember.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraGroupOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraServicePrincipalOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraTenantDetail.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUser.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserCreatedObject.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserDirectReport.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserOwnedDevice.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserOwnedObject.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserRegisteredDevice.ps1 ✅Succeeded

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link

learn-build-service-prod bot commented Sep 19, 2024

Learn Build status updates of commit d6d1f0e:

✅ Validation status: passed

File Status Preview URL Details
module/Entra/AdditionalFunctions/Find-EntraPermission.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAdministrativeUnit.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAdministrativeUnitMember.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraApplicationTemplate.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAuditDirectoryLog.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAuditSignInLog.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraObjectSetting.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraPolicy.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraApplication.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraApplicationOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraContact.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeletedApplication.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeviceRegisteredOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeviceRegisteredUser.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDirectoryRoleDefinition.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraGroupMember.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraGroupOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraServicePrincipalOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraTenantDetail.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUser.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserCreatedObject.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserDirectReport.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserOwnedDevice.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserOwnedObject.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserRegisteredDevice.ps1 ✅Succeeded

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link

learn-build-service-prod bot commented Sep 19, 2024

Learn Build status updates of commit 85cccc2:

✅ Validation status: passed

File Status Preview URL Details
module/Entra/AdditionalFunctions/Find-EntraPermission.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAdministrativeUnit.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAdministrativeUnitMember.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraApplicationTemplate.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAuditDirectoryLog.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAuditSignInLog.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraObjectSetting.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraPolicy.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraApplication.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraApplicationOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraContact.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeletedApplication.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeviceRegisteredOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeviceRegisteredUser.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDirectoryRoleDefinition.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraGroupMember.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraGroupOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraServicePrincipalOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraTenantDetail.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUser.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserCreatedObject.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserDirectReport.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserOwnedDevice.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserOwnedObject.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserRegisteredDevice.ps1 ✅Succeeded

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.

For any questions, please:

@v-varshamane v-varshamane added P0 Highest priority Ready For Review The PR Ready for Review labels Sep 19, 2024
@v-varshamane v-varshamane linked an issue Sep 19, 2024 that may be closed by this pull request
-Top parameter should not be defined as a nullable #1082
Closed
KenitoInc
KenitoInc requested changes Sep 24, 2024
View reviewed changes
@v-akarke v-akarke requested a review from KenitoInc September 24, 2024 08:41
@learn-build-service-prod
Copy link

learn-build-service-prod bot commented Sep 24, 2024

Learn Build status updates of commit fcf3ed5:

✅ Validation status: passed

File Status Preview URL Details
module/Entra/AdditionalFunctions/Find-EntraPermission.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAdministrativeUnit.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAdministrativeUnitMember.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraApplicationTemplate.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAuditDirectoryLog.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAuditSignInLog.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraObjectSetting.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraPolicy.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraApplication.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraApplicationOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraContact.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeletedApplication.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeviceRegisteredOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeviceRegisteredUser.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDirectoryRoleDefinition.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraGroupMember.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraGroupOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraServicePrincipalOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraTenantDetail.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUser.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserCreatedObject.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserDirectReport.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserOwnedDevice.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserOwnedObject.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserRegisteredDevice.ps1 ✅Succeeded

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link

learn-build-service-prod bot commented Sep 24, 2024

Learn Build status updates of commit 1029b7e:

✅ Validation status: passed

File Status Preview URL Details
module/Entra/AdditionalFunctions/Find-EntraPermission.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAdministrativeUnit.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAdministrativeUnitMember.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraApplicationTemplate.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAuditDirectoryLog.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraAuditSignInLog.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraObjectSetting.ps1 ✅Succeeded
module/Entra/AdditionalFunctions/Get-EntraPolicy.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraApplication.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraApplicationOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraContact.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeletedApplication.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeviceRegisteredOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDeviceRegisteredUser.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraDirectoryRoleDefinition.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraGroupMember.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraGroupOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraServicePrincipalOwner.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraTenantDetail.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUser.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserCreatedObject.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserDirectReport.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserOwnedDevice.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserOwnedObject.ps1 ✅Succeeded
module/Entra/customizations/Get-EntraUserRegisteredDevice.ps1 ✅Succeeded

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.

For any questions, please:

@KenitoInc KenitoInc merged commit 52a671d into main Sep 24, 2024
@v-varshamane v-varshamane mentioned this pull request Sep 24, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@KenitoInc KenitoInc KenitoInc approved these changes

@v-varshamane v-varshamane Awaiting requested review from v-varshamane

Assignees

No one assigned

Labels

P0 Highest priority Ready For Review The PR Ready for Review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

-Top parameter should not be defined as a nullable

4 participants

@v-akarke @KenitoInc @v-varshamane