Skip to content

Commit

Permalink
Add success messages for revoke membership in engagement form page (#…
Browse files Browse the repository at this point in the history
…2034)

* Revoke membership in user details page

* Fix linting

* Add include engagement param

* Create a full version of membership

* Make lintfix compatible with all OSes

* Fix flake8 issues

* rever query splitting

* update engagement_id check

* Add notif messages to revoke membership engagement form
  • Loading branch information
jadmsaadaot authored Aug 17, 2023
1 parent 166e915 commit 7b6ae28
Showing 1 changed file with 12 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,12 @@ export const ActionsDropDown = ({ membership }: { membership: EngagementTeamMemb
await revokeMembership(membership.engagement_id, membership.user_id);
loadTeamMembers();
setLoading(false);
dispatch(
openNotification({
text: `You have successfully revoked ${membership.user.first_name} ${membership.user.last_name} from ${membership.engagement?.name}`,
severity: 'success',
}),
);
} catch (error) {
setLoading(false);
dispatch(openNotification({ text: 'Failed to revoke membership', severity: 'error' }));
Expand All @@ -36,6 +42,12 @@ export const ActionsDropDown = ({ membership }: { membership: EngagementTeamMemb
await reinstateMembership(membership.engagement_id, membership.user_id);
loadTeamMembers();
setLoading(false);
dispatch(
openNotification({
text: `You have successfully reinstated ${membership.user.first_name} ${membership.user.last_name} on ${membership.engagement?.name}`,
severity: 'success',
}),
);
} catch (error) {
setLoading(false);
dispatch(openNotification({ text: 'Failed to reinstate membership', severity: 'error' }));
Expand Down

0 comments on commit 7b6ae28

Please sign in to comment.