Skip to content

Conversation

@joowojr
Copy link
Contributor

@joowojr joowojr commented Jan 30, 2025

📄 요약(Summary)

회원 상세 정보 조회 & 회원 정보 수정 API 구현

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • 알림 설정 데이터를 저장하는 member_notification 테이블 삭제

    • member table에 아지트/이메일/카카오 워크 알림 여부를 각각 저장하는 필드 추가
  • 회원 상세 정보 조회 API 구현

  • 회원 정보 수정 API 구현

    • 회원 정보의 이미지 수정 시에, multipart로 받은 파일이 이미지 형식인지 검증하는 util 추가
  • usecase 세분화

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

  • @starboxxxx 플랫폼 당 알림 수신여부에 대한 데이터에 따라 알림을 보낼 사용자를 필터링하는 로직 추가가 필요할거 같습니다!

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • PR 양식에 맞게 작성했습니다
  • 모든 테스트가 통과했습니다
  • 프로그램이 정상적으로 작동합니다
  • 적절한 PR 라벨을 설정했습니다
  • 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #164

<footer>
- 관련: #164
@joowojr joowojr added ✨ feature 구현·개선 사항에 관련된 내용입니다 HIGH 우선순위 상 labels Jan 30, 2025
@joowojr joowojr requested a review from starboxxxx January 30, 2025 11:17
@joowojr joowojr self-assigned this Jan 30, 2025
Copy link
Collaborator

@starboxxxx starboxxxx left a comment

Choose a reason for hiding this comment

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

Member entity에 있는 웹훅 컬럼에서 이제 활설화 비활성화 관리하는건가요?
각각에 대해 true인 것들만 알림 보내는걸로 구현하면 되는건가요??

@joowojr
Copy link
Contributor Author

joowojr commented Jan 30, 2025

Member entity에 있는 웹훅 컬럼에서 이제 활설화 비활성화 관리하는건가요? 각각에 대해 true인 것들만 알림 보내는걸로 구현하면 되는건가요??

네 맞습니다!

Copy link
Collaborator

@starboxxxx starboxxxx left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~

@joowojr joowojr merged commit 8594f3c into develop Jan 30, 2025
1 check passed
@joowojr joowojr deleted the CLAP-172 branch February 3, 2025 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feature 구현·개선 사항에 관련된 내용입니다 HIGH 우선순위 상

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-172 회원 상세정보 조회 및 수정 API 구현

4 participants