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

[Feat] 그룹 생성 페이지 API 연결 #21

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

ttaerrim
Copy link
Contributor

@ttaerrim ttaerrim commented Feb 25, 2024

설명

완료한 기능 명세

  • 그룹 생성 페이지 API 연결
    • service /group POST 추가
    • queries 그룹 생성 훅 추가
    • 그룹 생성 성공 시 모달 띄우고 모달 확인 버튼 클릭 시 그룹 리스트로 이동
      • 싱글 모달도 onClickConfirm 사용할 수 있도록 수정하였음

스크린샷

기능 작업에 대한 스크린샷/화면 녹화 있을 경우 첨부하기

리뷰 요청 사항

특별히 리뷰해 주었으면 하는 부분, 고민되는 부분 기재하기

const queryClient = useQueryClient();

return useMutation({
mutationFn: (id: string) => group.leave({ id }),
Copy link
Contributor

Choose a reason for hiding this comment

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

밑의 useLeaveGroupQuery()랑 함수 내용이 뒤바뀐 것 같습니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

감사합니다! 반영했습니다.

- 두 쿼리의 이름과 기능이 반대로 작성되어 있어 이를 변경함
<Radio
key={type}
id={type}
id={`${type}`}
Copy link
Contributor

Choose a reason for hiding this comment

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

p3 : 이 부분은 한 번만 감싸도 될 것 같습니다.

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

Successfully merging this pull request may close these issues.

[FEATURE] 그룹 생성 페이지 API 연결
3 participants