-
Notifications
You must be signed in to change notification settings - Fork 67
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
Refactor organization user sharing implementation and implement user sharing on sub-organization creation. #252
Refactor organization user sharing implementation and implement user sharing on sub-organization creation. #252
Conversation
305fdfc
to
1bede01
Compare
...rganization/management/organization/user/association/OrganizationUserAssociationService.java
Outdated
Show resolved
Hide resolved
...rganization/management/organization/user/association/OrganizationUserAssociationService.java
Outdated
Show resolved
Hide resolved
...rganization/management/organization/user/association/OrganizationUserAssociationService.java
Outdated
Show resolved
Hide resolved
e1c964f
to
4de9c6c
Compare
PR builder started |
PR builder completed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving the pull request based on the successful pr build https://github.com/wso2/product-is/actions/runs/6435267365
7eb469c
to
29438e4
Compare
PR builder started |
PR builder started |
PR builder completed |
PR builder completed |
...ty/organization/management/organization/user/sharing/OrganizationUserSharingServiceImpl.java
Outdated
Show resolved
Hide resolved
...ization/management/organization/user/sharing/internal/OrganizationUserSharingDataHolder.java
Outdated
Show resolved
Hide resolved
...anagement/organization/user/sharing/listener/SharingOrganizationCreatorUserEventHandler.java
Show resolved
Hide resolved
6f0e09a
to
2e8a334
Compare
...carbon/identity/organization/management/organization/user/sharing/constant/SQLConstants.java
Outdated
Show resolved
Hide resolved
...entity/organization/management/organization/user/sharing/OrganizationUserSharingService.java
Outdated
Show resolved
Hide resolved
...carbon/identity/organization/management/organization/user/sharing/constant/SQLConstants.java
Outdated
Show resolved
Hide resolved
...n/management/organization/user/sharing/internal/OrganizationUserSharingServiceComponent.java
Outdated
Show resolved
Hide resolved
...n/management/organization/user/sharing/internal/OrganizationUserSharingServiceComponent.java
Outdated
Show resolved
Hide resolved
...anagement/organization/user/sharing/listener/SharingOrganizationCreatorUserEventHandler.java
Outdated
Show resolved
Hide resolved
...rbon/identity/organization/management/role/management/service/dao/RoleManagementDAOImpl.java
Outdated
Show resolved
Hide resolved
...rganization/management/tenant/association/listeners/TenantAssociationManagementListener.java
Outdated
Show resolved
Hide resolved
...tity/organization/user/invitation/management/internal/UserInvitationMgtServiceComponent.java
Outdated
Show resolved
Hide resolved
...on/identity/organization/user/invitation/management/constant/UserInvitationMgtConstants.java
Show resolved
Hide resolved
.../wso2/carbon/identity/organization/user/invitation/management/InvitationCoreServiceImpl.java
Outdated
Show resolved
Hide resolved
…sharing on sub-organization creation.
e358626
to
9f3e375
Compare
Purpose
Depends on
Tested flows
Limitations
The carbon context lack the information of authenticated user's resident organization. The carbon context should be improved to have
userResidentOrganization
which can be set at the post authentication at valve level. Till that this workaround is done for make the feature work for super organization.Ref - https://github.com/wso2-extensions/identity-organization-management/pull/252/files#diff-4a2c3f0dc643c462a1cde55a741c1360d7803b08da9c35462fc504ddc7b64334R70-R72