save-my-youth icon indicating copy to clipboard operation
save-my-youth copied to clipboard

Fix: 청약 리스트 요청 API 변경

Open hotbreakb opened this issue 2 years ago • 1 comments

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

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

반영 브랜치

fix/call-apply-api -> dev

변경 사항

  • heroku에서 공공 API에 다이렉트로 요청합니다.
    • 기존에 백엔드를 거쳐 공공 API에 요청했던 이유는 vite에서 환경변수 접근하는 법을 몰라 시크릿키를 설정할 수 없었습니다. 이것을 해결하였습니다. .envVITE_APP_API_KEY = 블라블라를 적어주시면 됩니다. 관련 문서
  • 청약을 subscription에서 apply로 조금씩 변경 중입니다.

테스트 결과

  • 로딩 속도가 크게 개선되지 않았습니다. 좋아요 수를 청약 하나와 연결하면서 생긴 문제였습니다. 적용하는 부분 차후 개선하겠습니다.
  • 하나의 훅 안에서 하고 있는 일이 많아 분리 작업을 할 예정입니다.

hotbreakb avatar Oct 24 '22 13:10 hotbreakb

Deploy Preview for save-my-youth ready!

Name Link
Latest commit b7b4620de4e597be53260f72f21c42415f619952
Latest deploy log https://app.netlify.com/sites/save-my-youth/deploys/6356909ca4deb00008631544
Deploy Preview https://deploy-preview-293--save-my-youth.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar Oct 24 '22 13:10 netlify[bot]