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-425 비밀번호 재설정 경로에도 잠금 계정 여부를 확인하도록 수정 및 로그인 로직 수정 #555

Merged
merged 6 commits into from
Feb 14, 2025

Conversation

joowojr
Copy link
Contributor

@joowojr joowojr commented Feb 13, 2025

📄 요약(Summary)

비밀번호 재설정 경로에도 잠금 계정 여부를 확인하도록 수정 및 로그인 로직 수정

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • 비밀번호 재설정 경로에도 잠금 계정 여부를 필터단에서 확인하도록 수정하였습니다.
  • CLAP-415 회원 테이블 unique 인덱스 삭제 #544 회원 닉네임, 이메일에 대한 unique 인덱스 삭제에 대한 side effect 발생
    • 기존에는 nickname 및 email 필드에 로그인 시에 닉네임으로 회원을 조회했지만, db에 중복으로 닉네임이 올라가기 때문에 active한 상태의 회원을 조건으로 걸어 닉네임을 조회하도록 수정

Requirements for Reviewer

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

PR Log

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

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

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

✅ 체크리스트(Checklist)

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

🚪 이슈 번호(Issue numbers)

Closes #554

@joowojr joowojr added 🐛 bug 버그를 수정합니다 HIGH 우선순위 상 labels Feb 13, 2025
@joowojr joowojr self-assigned this Feb 13, 2025
@joowojr joowojr merged commit 3ab7468 into develop Feb 14, 2025
1 check passed
@joowojr joowojr added ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다. and removed 🐛 bug 버그를 수정합니다 labels Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HIGH 우선순위 상 ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다.
Projects
None yet
1 participant