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

Fix #117 자동 매칭방 탐색시 타입이 수동인 매칭방을 제외하고 탐색 #117

Merged
merged 2 commits into from
Feb 6, 2025

Conversation

hyxklee
Copy link
Member

@hyxklee hyxklee commented Feb 6, 2025

📌 관련 이슈

관련 이슈 번호 #116
Close #116

🚀 작업 내용

  • 수동 매칭방이 있는 상태에서 동일한 조건의 자동매칭을 돌렸을 때 수동 매칭방에 매칭이 되는 문제가 있어 이를 수정했습니다.
  • 해당 매칭 알고리즘이 임시인 만큼 위도 경도를 추가 검색하는 기존 자동매칭 로직에서는 해당 문제가 없을 것으로 예상합니다.
  • 하지만 수동 매칭 또한 일관성 있게, 혹은 사용자가 지도에 찍은 위치를 받아 위도 경도를 함께 저장하도록 확장되는 경우 DB에서 가져올 때 matchingRoomType을 적절하게 필터링해줘야할 듯 합니다.
  • 자동 방의 경우 생성될 때 Type이 들어가고 있지 않아 이를 추가했습니다.

📸 스크린샷

📢 리뷰 요구사항

@hyxklee hyxklee added the 💻 Fix 코드 수정 label Feb 6, 2025
@hyxklee hyxklee self-assigned this Feb 6, 2025
@@ -31,8 +32,10 @@ public interface MatchingRoomRepository extends JpaRepository<MatchingRoom, Long
@Query("SELECT r FROM MatchingRoom r " +
"WHERE r.departure = :departure " +
"AND r.destination = :destination " +
"AND r.matchingRoomStatus = 'ACTIVE' ")
"AND r.matchingRoomStatus = 'ACTIVE'" +
"AND r.matchingRoomType = 'AUTO' ")
Copy link
Member

Choose a reason for hiding this comment

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

고생하셨습니다 !

변경된 내용 확인했습니당

@hyxklee hyxklee merged commit f6030a5 into dev Feb 6, 2025
2 checks passed
@hyxklee hyxklee deleted the fix/#116/매칭방-검색 branch February 6, 2025 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💻 Fix 코드 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix]: #116 자동 매칭방 탐색시 수동 매칭방은 제외하도록 수정
3 participants