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

Refactor #107 departuretime 변수형 및 자동매칭 수정 #107

Merged

Conversation

huncozyboy
Copy link
Member

@huncozyboy huncozyboy commented Feb 5, 2025

📌 관련 이슈

관련 이슈 번호 #101
Close #

🚀 작업 내용

  • departureTime 변수형 String으로 모두 변경
  • 수동매칭 나의 매칭 리스트도 active 상태인 방만 조회되도록 변경
  • 자동매칭 입력 받는 필드에서 위도경도 제외 후 , 알고리즘 돌릴 때 출발, 도착지만 검색해서 필터링 되도록 수정

자동매칭 관련은
refactor : 자동매칭 위도, 경도 사용안하도록 수정 이라는 커밋으로 한번에 자동매칭 수정된거 올려서 해당 커밋 누르고 파일체인지 봐주시면 구별 잘될거같아여

📸 스크린샷

image
구독 후
image
출발지 도착지로 자동매칭 요청


아래는 DB에 추가된 필드들 정상적으로 들어간거 캡쳐 화면이라, 같이 확인해주시면 될거같습니다 !

image

image

image

image

📢 리뷰 요구사항

테스트까지 완료했는데 고친 부분이 많고 제 파트가 아니라, 빠진 부분이 있을 수도 있을거같아서 꼼꼼히 확인해주시면 감사하겠습니다

@huncozyboy huncozyboy added the 🔨 Refactor 코드 리팩토링 label Feb 5, 2025
@huncozyboy huncozyboy self-assigned this Feb 5, 2025
@huncozyboy huncozyboy linked an issue Feb 5, 2025 that may be closed by this pull request
3 tasks
Copy link
Member

@hyxklee hyxklee left a comment

Choose a reason for hiding this comment

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

제가 봤을 때 문제는 없는 것 같아요!
후에 다시 위도 경도 도입할 때 해당 PR 내용으로 원복하면 될 것 같습니다!

Copy link
Collaborator

@soyesenna soyesenna left a comment

Choose a reason for hiding this comment

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

고생하셨서용
확인했습니다!!

@huncozyboy huncozyboy merged commit 678bc53 into dev Feb 5, 2025
2 checks passed
@huncozyboy huncozyboy deleted the refactor/#101/departuretime-변수형-및-자동매칭-수정 branch February 6, 2025 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor]: #101 departureTime 변수형 및 자동매칭 수정
3 participants