Skip to content
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

fix: 초대링크를 통한 가입시 storeId 받아올 수 있게 로직 변경 #98

Conversation

iOdiO89
Copy link
Member

@iOdiO89 iOdiO89 commented Jul 12, 2024

#97

@iOdiO89 iOdiO89 requested a review from DeveloperRyou July 12, 2024 14:01
@iOdiO89 iOdiO89 force-pushed the 97-직원초대링크-생성과-가입-플로우를-안정화한다 branch from 47a1075 to 71e9648 Compare July 12, 2024 14:12
@iOdiO89 iOdiO89 force-pushed the 97-직원초대링크-생성과-가입-플로우를-안정화한다 branch from 865c7a3 to e917a5b Compare July 12, 2024 14:13
@iOdiO89 iOdiO89 changed the title fix: 초대링크를 통한 가입시 storeId 받아올 수 있게 로직 변경 #97 fix: 초대링크를 통한 가입시 storeId 받아올 수 있게 로직 변경 Jul 12, 2024
@iOdiO89 iOdiO89 changed the title #97 fix: 초대링크를 통한 가입시 storeId 받아올 수 있게 로직 변경 # fix: 초대링크를 통한 가입시 storeId 받아올 수 있게 로직 변경 Jul 12, 2024
@iOdiO89 iOdiO89 changed the title # fix: 초대링크를 통한 가입시 storeId 받아올 수 있게 로직 변경 #{97} fix: 초대링크를 통한 가입시 storeId 받아올 수 있게 로직 변경 Jul 12, 2024
@iOdiO89 iOdiO89 changed the title #{97} fix: 초대링크를 통한 가입시 storeId 받아올 수 있게 로직 변경 fix: 초대링크를 통한 가입시 storeId 받아올 수 있게 로직 변경 Jul 12, 2024
@iOdiO89
Copy link
Member Author

iOdiO89 commented Jul 12, 2024

아직 머지하지 말아주세요! postRelation이 두번씩 호출되는 것 같아서 잠시 확인해보겠습니다

@iOdiO89 iOdiO89 force-pushed the 97-직원초대링크-생성과-가입-플로우를-안정화한다 branch from 3a22453 to cbafc74 Compare July 12, 2024 17:53
@iOdiO89 iOdiO89 force-pushed the 97-직원초대링크-생성과-가입-플로우를-안정화한다 branch from cbafc74 to 8a6158c Compare July 12, 2024 18:29
@iOdiO89
Copy link
Member Author

iOdiO89 commented Jul 12, 2024

dynamotest 브랜치 삭제해서 amplify에서도 브랜치 분리하려고 했는데 권한이 없네요. 브랜치 분리 부탁드립니다!
앱설정 > 브랜치 설정에서 dynamo-test 브랜치 선택 후 작업 > 브랜치 분리해주시면 됩니다

image

그리고 정확하진 않지만, 저번에 dynamo-test 브랜치를 배포하면서 amplifyapp.com이 새로 생긴 것 같아서요..!
아닐수도 있지만 혹시나 요금이 더 나갈까봐 .. 말씀드립니다..!!

image

Copy link
Contributor

@DeveloperRyou DeveloperRyou left a comment

Choose a reason for hiding this comment

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

확인했습니다. 현재 아직도 PostPlan은 작동하지 않고 있나요?

Comment on lines +72 to +86
const { mutateAsync: postRelationMutate } = usePostRelation();
const { mutate: postPlanListMutate } = usePostPlanList();
const queryClient = useQueryClient();
const router = useRouter();

const { mutate } = useMutation({
mutationKey: ["StaffJoin"],
mutationFn: async ({
storeId,
memberId,
body,
inviteSchedule,
}: UsePostRelationProps & { inviteSchedule: InviteSchedule }) => {
await postRelationMutate({ storeId, memberId, body });
postPlanListMutate({ storeId, memberId, inviteSchedule });
Copy link
Contributor

Choose a reason for hiding this comment

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

mutateAsync라는게 있었네요.. 이걸 알았으면 더 빨리 되었을것 같습니다.
postPlanListMutate도 await 걸 수 있을거 같아요

@DeveloperRyou DeveloperRyou merged commit 7507fcb into the-kingdoms:develop Jul 13, 2024
1 check passed
@iOdiO89
Copy link
Member Author

iOdiO89 commented Jul 13, 2024

아니요 잘 작동합니다!
이제 직원 초대 플로우 전부 정상적입니다 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants