Skip to content

Conversation

Chan531
Copy link
Contributor

@Chan531 Chan531 commented Jan 24, 2025

✨ Related Issue

📝 기능 구현 명세

  • 테스트 진행 X

🐥 추가적인 언급 사항

  • Member와 Auth 간에 api 재분배가 필요할 거 같습니다. 개인적으로 인증/인가 관련 로그인, 회원가입, 토큰 재발급을 Auth로, 이미 생성된 멤버에 대한 로직인 비밀번호 변경, 회원 탈퇴를 Member에 두는 것이 좋은 거 같습니다.
  • 테스트 따로 다음주 중에 진행 후, 머지하겠습니다.

@Chan531 Chan531 added the feat 기능 label Jan 24, 2025
@Chan531 Chan531 requested a review from paragon0107 January 24, 2025 06:07
@Chan531 Chan531 self-assigned this Jan 24, 2025
Copy link
Contributor

@paragon0107 paragon0107 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 리뷰 확인 후 지금 즉시 반영되어야 할 필요가 있는것들만 수정 후 머지 부탁드릴게용

@paragon0107 paragon0107 merged commit 7ae19db into develop Jan 29, 2025
1 check passed
@paragon0107 paragon0107 deleted the feat/#243-delete-api branch January 29, 2025 08:15
@paragon0107
Copy link
Contributor

말씀해주신 API에 대해서 회원가입은 멤버 도메인에 두는게 어떨까요? 따로 인증관련 로직을 진행하지 않고 단순히 멤버 생성이니까 멤버도메인으로 가는게 좋을것 같은게 개인적인 생각입니당

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

Labels

feat 기능

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] 삭제 api 수정

2 participants