Skip to content
This repository was archived by the owner on Jan 11, 2026. It is now read-only.

Conversation

@msk226
Copy link
Member

@msk226 msk226 commented Apr 13, 2025

#️⃣ 연관된 이슈

#411

🔎 작업 내용

  1. 내 관심 지역, 관심 분야, 모집 중 스터디 조회 시, 활동비를 최댓값만 입력 받는게 아닌, 범위로 필터링 할 수 있도록 로직 확장

📷 스크린샷 (선택)

작업한 결과물에 대한 간단한 스크린샷을 올려주세요.


💬리뷰 요구사항 (선택)

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

@msk226 msk226 added the 👍 feature New feature or request label Apr 13, 2025
@msk226 msk226 requested review from FromKyoung and dvlp-sy April 13, 2025 14:26
@msk226 msk226 self-assigned this Apr 13, 2025
Copy link
Collaborator

@dvlp-sy dvlp-sy left a comment

Choose a reason for hiding this comment

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

코드 읽다 보니까 저희 지금 전체적으로 성별 조건에 대한 검증 로직이 없는 것 같아요...!!
(제가 담당한 스터디 생성/참여 부분에도 성별 조건 확인 로직을 추가해야 할 것 같아요)

@msk226
Copy link
Member Author

msk226 commented Apr 14, 2025

코드 읽다 보니까 저희 지금 전체적으로 성별 조건에 대한 검증 로직이 없는 것 같아요...!! (제가 담당한 스터디 생성/참여 부분에도 성별 조건 확인 로직을 추가해야 할 것 같아요)

이렇게 적절한 안내 메세지를 던질 수 있도록 해놨습니다!

잘못된 성별이 들어오면 Enum 변환 과정에서 에러가 발생 했다는 응답이 전달되긴 하는데, 가독성이 좀 떨어져서 해당 부분은 제가 커스터마이징 했어요!

스크린샷 2025-04-14 오전 9 32 51

@dvlp-sy
Copy link
Collaborator

dvlp-sy commented Apr 14, 2025

입력 조건 외의 회원의 성별과 스터디가 요구하는 성별을 대조할 수 있는 로직이 필요할 것 같습니다..!

@msk226
Copy link
Member Author

msk226 commented Apr 14, 2025

입력 조건 외의 회원의 성별과 스터디가 요구하는 성별을 대조할 수 있는 로직이 필요할 것 같습니다..!

앗 제가 이해를 잘못했군요 ㅋㅋㅋㅋㅋ

@msk226
Copy link
Member Author

msk226 commented Apr 14, 2025

입력 조건 외의 회원의 성별과 스터디가 요구하는 성별을 대조할 수 있는 로직이 필요할 것 같습니다..!

이건 이슈 따로 생성해서 반영하겠습니다!

++ PM님한테 이 부분 여쭤볼게요! 저번에 스터디 성별은 그냥 참고값이라고 들었던 것 같아서요!

@msk226 msk226 merged commit a13acf5 into develop Apr 14, 2025
1 check passed
@msk226 msk226 deleted the SPOT-268/modify branch April 14, 2025 09:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

👍 feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] 스터디 조회 시, 활동비를 범위로 필터링 하도록 수정

3 participants