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

datamap [♻️ Refactoring] : 핀 클릭, 상세보기, 스크랩 , 장소 목록 #24

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from

Conversation

moonyaeyoon
Copy link
Contributor

@moonyaeyoon moonyaeyoon commented Aug 18, 2023

PR 체크리스트

  • PR
  • PR

PR 타입

  • 🐞 Bugfix
  • ✨ Feature
  • ♻️ Refactoring
  • 📄 Documentation content changes
  • 🎨 Style
  • 🧱 Other... Please describe:

Test 방법

node index.js

To. 리뷰어

기타사항

  1. 12시간 전 혼잡도는 공공데이터 출력값에 없어서 구현 고민 중입니다... DB에 시간대별로 저장해두고 업데이트 하는 방식...?으로 해야할까요
  2. 출력값에 있는 인근 장소가 혼잡도 장소와 생각보다 멀어서 주소를 직접 DB에 넣을지 , 프론트 측에 부탁할지 고민입니다!

@moonyaeyoon moonyaeyoon changed the title Api/#2 datamap datamap [♻️ Refactoring] : 핀 클릭, 상세보기, 스크랩 , 장소 목록 Aug 18, 2023
@moonyaeyoon moonyaeyoon added the ✨feat 새로운 기능 구현 label Aug 18, 2023
@moonyaeyoon moonyaeyoon self-assigned this Aug 18, 2023
@moonyaeyoon moonyaeyoon added the 😃예닝 예닝/문예윤 label Aug 18, 2023
@jyjyjy25
Copy link
Contributor

  1. 아마 저장해두고 제일 혼잡했던 시간대를 비교해서 가져와야 할 것 같은데.. 그럼 장소에 대한 추이 데이터를 저장하는 테이블을 하나 추가해서 공공데이터를 받아올 때마다 해당 테이블에 시간대와 같이 혼잡도 정보를 추가하는 방식으로 구현해야 하지 않을까 싶네요..! 그리고 12시간이 지난 데이터는 테이블에서 삭제하는 로직도 있어야 할 것 같구요 !
  2. 출력값에 있는 인근 장소라는게 공공데이터 출력값 중 인근 장소를 말씀하시는 게 맞을까요 ??
  3. 아 그리고 충돌난 파일이 몇 개 있는데 해당 파일들 충돌 해결해 주셔야 할 것 같아요!

@moonyaeyoon
Copy link
Contributor Author

네,,! 출력값에 있는 인근 장소입니다!
파일 충돌 해결하면서 말씀해주신 방법으로 12시간 전 ㅓ데이터도 구현해보겠습니다,,!

@moonyaeyoon
Copy link
Contributor Author

아 지금 확인해보니까 제 파일에서는 충돌이 없는데 혹시 어디서 충돌나셨을까용,,,?

@jyjyjy25
Copy link
Contributor

jyjyjy25 commented Aug 27, 2023

엇 바로 위에 conflict났다고 표시되는 파일들이 있어서요!

@moonyaeyoon
Copy link
Contributor Author

아 dev 파일이랑 제 파일이랑 중복되었었더라구요,,! 해결했습니다,,!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨feat 새로운 기능 구현 😃예닝 예닝/문예윤
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants