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

Add organization shared user listener #231

Merged

ShanChathusanda93 merged 1 commit into wso2-extensions:main from ShanChathusanda93:invited-org-user-listener-branch

Aug 28, 2023

Contributor

ShanChathusanda93 commented Aug 23, 2023

Purpose

$subject
In this initial implementation, when a user is deleted then if there are any associations to the child organizations, they will be removed before deleting the user.

ShanChathusanda93 force-pushed the invited-org-user-listener-branch branch 2 times, most recently from 3b77540 to b98ae20 Compare

August 25, 2023 11:15

thanujalk reviewed

View reviewed changes

...ty/organization/user/invitation/management/listener/OrgSharedUserOperationEventListener.java Show resolved Hide resolved

sadilchamishka reviewed

View reviewed changes

.../org/wso2/carbon/identity/organization/user/invitation/management/InvitationCoreService.java Outdated Show resolved Hide resolved

sadilchamishka reviewed

View reviewed changes

.../org/wso2/carbon/identity/organization/user/invitation/management/InvitationCoreService.java Outdated Show resolved Hide resolved


          Add organization shared user listener

b6d0c7b

ShanChathusanda93 force-pushed the invited-org-user-listener-branch branch from b98ae20 to b6d0c7b Compare

August 28, 2023 06:28

inthirakumaaran previously approved these changes

View reviewed changes

sadilchamishka reviewed

View reviewed changes

.../org/wso2/carbon/identity/organization/user/invitation/management/InvitationCoreService.java Show resolved Hide resolved

sadilchamishka reviewed

View reviewed changes

.../org/wso2/carbon/identity/organization/user/invitation/management/dao/UserInvitationDAO.java Show resolved Hide resolved

sadilchamishka reviewed

View reviewed changes

.../org/wso2/carbon/identity/organization/user/invitation/management/constant/SQLConstants.java Show resolved Hide resolved

sadilchamishka reviewed

View reviewed changes

.../org/wso2/carbon/identity/organization/user/invitation/management/constant/SQLConstants.java Show resolved Hide resolved

inthirakumaaran dismissed their stale review

August 28, 2023 07:55

Address the above comments

Contributor

sadilchamishka commented Aug 28, 2023

Discussed about the above raised comments and agree to proceed as those suggessions were improvements to be done after discussion/review with the team. Hence approving the PR.

sadilchamishka approved these changes

View reviewed changes

ShanChathusanda93 merged commit a714ae4 into wso2-extensions:main

1 check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet