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

Building 위경도 정보 같이 내려주기 #233

Merged
merged 2 commits into from
Mar 3, 2024

Conversation

sanggggg
Copy link
Contributor

@sanggggg sanggggg commented Mar 2, 2024

Checklist

  • 충분한 양의 자동화 테스트를 작성했는가?
    • 계단정복지도 서비스는 사이드 프로젝트로 진행되는 만큼 충분한 QA 없이 배포되는 경우가 많습니다. 따라서 자동화 테스트를 꼼꼼하게 작성하는 것이 서비스 품질을 유지하는 데 매우 중요합니다.

지도 UI 보여주기를 위해 장소(건물) 위경도 정보가 필요하다

현재 위경도 정보를 내려주는 DTO 가 Building (== PlaceListItem) 뿐이고,
이걸 사용하는 엔드포인트는 searchPlace, listConqueredPlaces, listPlaceInBuilding

정도여서 위경도 같이 내려주는게 큰 문제는 없어보인다

@sanggggg sanggggg requested a review from a team as a code owner March 2, 2024 15:43
Copy link

github-actions bot commented Mar 2, 2024

🔥🔥🔥 Backend CI Failed. github action link 🔥🔥🔥

@sanggggg sanggggg merged commit 429762e into main Mar 3, 2024
2 checks passed
@sanggggg sanggggg deleted the sanggggg/building-add-latlng branch March 3, 2024 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants