Skip to content

[CHORE] 구현테스트 위한 코드 수정#18

Open
soso0023 wants to merge 7 commits into
devfrom
api/#7_pinview
Open

[CHORE] 구현테스트 위한 코드 수정#18
soso0023 wants to merge 7 commits into
devfrom
api/#7_pinview

Conversation

@soso0023

@soso0023 soso0023 commented Aug 7, 2023

Copy link
Copy Markdown
Contributor

PR 체크리스트

  • PR
  • PR

PR 타입

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

Test 방법

To. 리뷰어

기타사항

@moonyaeyoon moonyaeyoon self-requested a review August 18, 2023 00:50

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

혹시 insertNearByPinQuery 에서 pinView 테이블의 createdAt도 nearByPin 테이블에 추가해주실 수 있을까용??

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

추가했습니다 !! 핀 생성 api 에서 핀이 테이블에 저장되면서 1km 이내인 핀도 인근핀 테이블에 저장되도록 해놨으니 필요할 때 조회만 하시면 될 것 같아요!!

Comment thread src/app/PinView/pinViewController.js
@jyjyjy25 jyjyjy25 self-requested a review August 27, 2023 15:01
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.

3 participants