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

[feat] load Location Info from OpenApi #25

Closed
wants to merge 3 commits into from

Conversation

parking0
Copy link
Contributor

@parking0 parking0 commented Jul 17, 2024

#️⃣연관된 이슈

ex) #26

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 기능 변경
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

반영 브랜치

ex) develop -> dev

📝작업 내용

공공데이터 OpenApi에서 서울시 음식점, 문화공간 데이터를 불러오도록 했습니다.

변경 사항

  1. json-simple gradle을 추가했습니다.
  2. 아직 DB가 제대로 구축되어 있지 않은 관계로, 서버 시작시 CommandLineRunner를 이용하여, openapi에서 데이터를 불러오도록 했습니다. 따라서 시간이 많이 소요되기 때문에, 데이터가 많은 음식점은 주석처리하고, 문화공간 데이터만 load하도록 했습니다. 아직 Location이 DB에 저장되도록 하지 않았습니다!!!

테스트 결과

서울시 문화공간 데이터가 모두 제대로 들어옵니다.

스크린샷 (선택)

image

💬리뷰 요구사항(선택)

AWS가 구축된 뒤에 음식점, 문화공간 데이터가 저장되도록 하겠습니다. (매번 서버 시작할 때마다 openapi에서 load해오면 시간이 많이 걸리기 때문입니다...) 다른 좋은 방법 있으면 말해주세요~

@parking0 parking0 self-assigned this Jul 17, 2024
@parking0 parking0 closed this Jul 22, 2024
@parking0 parking0 deleted the develop branch July 22, 2024 09:15
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.

1 participant