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

Feature/#380 팀 및 스터디 탈퇴 #381

Merged
merged 13 commits into from
Jan 26, 2025

Conversation

yeonddori
Copy link
Collaborator

관련 이슈

작업 요약

  • 팀 및 스터디 탈퇴 버튼을 구현하였습니다.

작업 상세 설명

  • 기존의 ControlPanel을 이용하여 그 자리에 탈퇴 버튼을 구현하였습니다.
  • api 동작은 아직 확인을 못 해봤습니다. 추후에 확인해 본 후 다시 수정하겠습니다!
  • 스터디 페이지는 권한 부분을 다 주석처리하고 확인해봐도 어떤 이유에서인지 자꾸 페이지가 몇 초 뒤에 사라져 스터디 탈퇴 버튼은 제대로 뜨는지 아직 확인해보지 못했습니다. 나중에 api 확인할 때 같이 확인해보겠습니다!

리뷰 요구 사항

  • 처음에는 삭제와 탈퇴 모달이 매우 유사해 하나의 모달로 통합하여 구현했었습니다. 그러나 스터디 작업을 진행하면서, 스터디 모달은 삭제와 종료가 별도로 나누어 구현되어 있는 것을 확인했습니다.. 따라서 팀 모달도 다시 통합된 구조를 풀고, 삭제와 탈퇴 모달을 각각 따로 구현하였습니다.

미리 보기

image

@yeonddori yeonddori added the ✨ 기능 개발 새로운 기능을 구현하는 데 필요한 작업 또는 변경 사항 label Jan 12, 2025
@yeonddori yeonddori self-assigned this Jan 12, 2025
Copy link
Collaborator

@llddang llddang left a comment

Choose a reason for hiding this comment

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

image

이유는 모르겠지만 백엔드 서버에서 405를 띄우고 있습니다.
한 번 다시 살펴보시고, 백엔드 분이랑 얘기해보셔야 할 것 같아요!

삭제와 탈퇴 모달을 각각 따로 구현한 것 굿입니당!!

@yeonddori
Copy link
Collaborator Author

image 이유는 모르겠지만 백엔드 서버에서 405를 띄우고 있습니다. 한 번 다시 살펴보시고, 백엔드 분이랑 얘기해보셔야 할 것 같아요!

삭제와 탈퇴 모달을 각각 따로 구현한 것 굿입니당!!

이것도 서버 설정을 수정한 이후로 이제 제대로 작동합니다! 감사합니당 ( ˶ˆᗜˆ˵ )

@yeonddori yeonddori requested a review from llddang January 26, 2025 11:25
Copy link
Collaborator

@jasper200207 jasper200207 left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Collaborator

@llddang llddang left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!

@yeonddori yeonddori merged commit 3e904f0 into develop Jan 26, 2025
1 check passed
@yeonddori yeonddori deleted the Feature/#380-팀_및_스터디_탈퇴 branch January 26, 2025 13:18
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