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

CLAP-362 회원 이메일 전송 시 상태 반영 #462

Merged
merged 3 commits into from
Feb 11, 2025
Merged

Conversation

Sihun23
Copy link
Collaborator

@Sihun23 Sihun23 commented Feb 10, 2025

📄 요약(Summary)

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • 회원 이메일 전송 후 MemberStatus 변경이 DB에 반영되지 않는 오류 해결
  • console log 제거

Requirements for Reviewer

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

PR Log

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

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

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

✅ 체크리스트(Checklist)

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

🚪 이슈 번호(Issue numbers)

Closes #459

@Sihun23 Sihun23 self-assigned this Feb 10, 2025
@joowojr joowojr added the 🏷️ release 릴리즈 관련 label Feb 10, 2025
@joowojr joowojr changed the title CLAP-362 회원 이메일 전송 CLAP-362 회원 이메일 전송 시 상태 반영 Feb 10, 2025
@joowojr joowojr changed the title CLAP-362 회원 이메일 전송 시 상태 반영 CLAP-362 회원 이메일 전송 Feb 10, 2025
@joowojr joowojr changed the title CLAP-362 회원 이메일 전송 CLAP-362 회원 이메일 전송 시 상태 반영 Feb 10, 2025
Copy link
Contributor

@joowojr joowojr left a comment

Choose a reason for hiding this comment

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

추가로 회원 전체 조회에 회원 상태도 반영 부탁드릴게요!

@Sihun23 Sihun23 requested a review from joowojr February 11, 2025 00:29
@Sihun23 Sihun23 requested a review from joowojr February 11, 2025 01:20
@joowojr joowojr merged commit 12feb9e into release Feb 11, 2025
1 check passed
@joowojr joowojr deleted the CLAP-362 branch February 11, 2025 01:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ release 릴리즈 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants