Skip to content

Commit

Permalink
Merge pull request #1966 from makeplane/fix/workspace-members-mutate
Browse files Browse the repository at this point in the history
fix: workspace members mutate issue
  • Loading branch information
sriramveeraghanta committed Aug 24, 2023
2 parents 7fca01d + bce8cae commit 489ef6a
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ type Props = {
setIsOpen: React.Dispatch<React.SetStateAction<boolean>>;
workspace_slug: string;
user: ICurrentUserResponse | undefined;
onSuccess: () => void;
};

type EmailRole = {
Expand All @@ -45,12 +46,8 @@ const defaultValues: FormValues = {
],
};

const SendWorkspaceInvitationModal: React.FC<Props> = ({
isOpen,
setIsOpen,
workspace_slug,
user,
}) => {
const SendWorkspaceInvitationModal: React.FC<Props> = (props) => {
const { isOpen, setIsOpen, workspace_slug, user, onSuccess } = props;
const {
control,
reset,
Expand Down Expand Up @@ -88,6 +85,7 @@ const SendWorkspaceInvitationModal: React.FC<Props> = ({
title: "Success!",
message: "Invitations sent successfully.",
});
onSuccess();
})
.catch((err) => {
setToastAlert({
Expand Down
10 changes: 8 additions & 2 deletions apps/app/pages/[workspaceSlug]/settings/members.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,10 @@ const MembersSettings: NextPage = () => {

const currentUser = workspaceMembers?.find((item) => item.member?.id === user?.id);

const handleInviteModalSuccess = () => {
mutateInvitations();
};

return (
<WorkspaceAuthorizationLayout
breadcrumbs={
Expand Down Expand Up @@ -139,8 +143,9 @@ const MembersSettings: NextPage = () => {
});
})
.finally(() => {
mutateMembers((prevData: any) =>
prevData?.filter((item: any) => item.id !== selectedRemoveMember)
mutateMembers(
(prevData: any) =>
prevData?.filter((item: any) => item.id !== selectedRemoveMember)
);
});
}
Expand Down Expand Up @@ -180,6 +185,7 @@ const MembersSettings: NextPage = () => {
setIsOpen={setInviteModal}
workspace_slug={workspaceSlug as string}
user={user}
onSuccess={handleInviteModalSuccess}
/>
<div className="p-8">
<SettingsHeader />
Expand Down

1 comment on commit 489ef6a

@vercel
Copy link

@vercel vercel bot commented on 489ef6a Aug 24, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Successfully deployed to the following URLs:

plane-dev – ./apps/app

plane-dev.vercel.app
plane-dev-plane.vercel.app
plane-dev-git-develop-plane.vercel.app

Please sign in to comment.