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

[필독]전달사항 #131

Open
alreadynyeong opened this issue Jan 8, 2024 · 0 comments · Fixed by #132
Open

[필독]전달사항 #131

alreadynyeong opened this issue Jan 8, 2024 · 0 comments · Fixed by #132
Assignees

Comments

@alreadynyeong
Copy link
Member

alreadynyeong commented Jan 8, 2024

제가 자고있을 경우를 대비해 이슈로 전달합니다!

💻 태영님이 해야할 것

PR 리뷰 달고 오류 없을 시 머지

  • 바꾼게 많으니 잘 확인부탁드립니다.

구현중이시던 기능

  • 하시던 기능 구현은 이어서 해주세요!

스타일 수정

  • 온보딩 화면이 디바이스 길이가 짧을 경우 버튼을 볼 수가 없습니다!
  • 로그인 버튼
  • 피그마 보면서 빠진 부분 확인

좋아요, 스크랩

  • 상세보기 외에 좋아요와 스크랩 버튼이 있는 경우 API 연동
스크린샷 2024-01-09 오전 5 14 33

🚨 오류

실행을 시켰을 때

Error: [vite]: Rollup failed to resolve import "@tanstack/react-query" from

이런 오류가 난다면 react-query 버전문제

  • Lock 파일과 node.moudules 삭제
  • 터미널에 yarn 입력
  • 다시 실행

🙂 하면 더 좋은 것

react-query

  • 연동시 useMutation or useQuery 등 사용

src/apis

  • Api 관련 코드 폴더에 정리하기
  • 해당 코드가 어떤 api인지 주석 달아주기

src/types

  • Type관련 코드 폴더에 정리해서 사용
  • 컴포넌트가 아닌 자주 사용되는 props일 경우 정리
  • ex) interface 어쩌고Props

src/constants

  • 서버에서 받아오는 데이터가 아닌 경우 폴더에 정리해서 사용
  • ex)WriteDetail.tsx 파일 속 const options = ['공식홈페이지', '쿠팡', '학생복지스토어', '기타'];
@alreadynyeong alreadynyeong linked a pull request Jan 8, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants