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

[FE] 지출 생성 시 나갔던 멤버를 다시 추가할 수 없는 오류 #637

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

Todari
Copy link
Contributor

@Todari Todari commented Sep 26, 2024

issue

구현 목적

지출 생성 시 지워졌던 멤버를 다시 추가할 수 없는 오류가 발생합니다.
예를 들어 1차에 이상, 감자, 망쵸 가 있을 때, 2차에서는 망쵸가 나가고 이상, 감자가 남습니다.
3차에서 망쵸를 다시 추가하려고 할 때, "�현재 참여하고 있는 인원이 존재합니다" 에러가 발생합니다.

2024-09-25.1.51.58.mov

구현 내용

기존에는 새로 멤버를 추가할 때, 이미 있는 멤버인지 판별하기 위해 currentMember와 비교했었습니다.

// useMembersStep.tsx
// ...
    const setBillInfoMemberWithId = (name: string) => {
    const existingMember = currentMembers.find(currentMember => currentMember.name === name);
    if (existingMember) {
      setBillInfo(prev => ({...prev, members: [...prev.members, {id: existingMember.id, name: name}]}));
    } else {
      setBillInfo(prev => ({...prev, members: [...prev.members, {id: -1, name: name}]}));
    }
  };
// ...

하지만, 한번 나갔던 멤버의 정보는 currentMember에 존재하지 않고, allMember에 존재하기 때문에 비교하는 대상을 currentMember에서 allMember로 변경해 주었습니다.

// useMembersStep.tsx
// ...
    const setBillInfoMemberWithId = (name: string) => {
    const existingMember = allMembers.find(currentMember => currentMember.name === name);
    if (existingMember) {
      setBillInfo(prev => ({...prev, members: [...prev.members, {id: existingMember.id, name: name}]}));
    } else {
      setBillInfo(prev => ({...prev, members: [...prev.members, {id: -1, name: name}]}));
    }
  };
// ...

@Todari Todari added 🖥️ FE Frontend 🚨 bug bug 🔍 QC quality check labels Sep 26, 2024
@Todari Todari added this to the v2.0.0 milestone Sep 26, 2024
@Todari Todari self-assigned this Sep 26, 2024
Copy link

Copy link
Contributor

@soi-ha soi-ha left a comment

Choose a reason for hiding this comment

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

멋져요 굳굳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚨 bug bug 🖥️ FE Frontend 🔍 QC quality check
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

[FE] 지출 생성 시 나갔던 멤버를 다시 추가할 수 없는 오류
3 participants