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

그룹 탈퇴를 할 수 있다. #167

Merged
merged 1 commit into from
Aug 13, 2024
Merged

그룹 탈퇴를 할 수 있다. #167

merged 1 commit into from
Aug 13, 2024

Conversation

Hwanvely
Copy link
Collaborator

변경 사항

AS-IS

GroupRequest에서 유효성 검사를 하여 null을 받을 수 없었습니다.

TO-BE

  1. null을 허용하기 위해 검사를 더이상 진행하지 않습니다
  2. null일 경우 그룹 탈퇴를 진행합니다.

@Hwanvely Hwanvely added the feat label Aug 13, 2024
@Hwanvely Hwanvely requested a review from 0chil August 13, 2024 10:42
@Hwanvely Hwanvely self-assigned this Aug 13, 2024
@0chil 0chil merged commit 288aa23 into dev Aug 13, 2024
1 check passed
@nijuy
Copy link

nijuy commented Aug 13, 2024

여기 잘....... 안되길래................ 질문 왔습니다 흑흑 (움짤이라 카톡보다 코멘트가 보기 편할거 같아서!!)

withdraw

{ group: null }로 보내면 400 에러 (페이로드를 읽지 못했다) 가 뜨는 상황인데요,
혹시 짐작 가는 무언가................ 있으싱가요.................................

@Hwanvely
Copy link
Collaborator Author

현재 null 처리가 안되어있어서 그럴겁니다.. ㅠㅠ 빨리 고쳐두겠습니다!

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

Successfully merging this pull request may close these issues.

3 participants