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

[ ViewProduct ] 코드 리팩토링 #164

Open
wants to merge 12 commits into
base: develop
Choose a base branch
from
Open

Conversation

say-young516
Copy link
Member

@say-young516 say-young516 commented Nov 29, 2022

🔥 Related Issues

🎡 작업 내용

  • 서버 통신 부분 확인 (캐싱 등.. )
  • recoil 지저분한 것 모두 selector로 만들어버리자
  • 반복적인 부분 함수 처리
  • 불필요한 코드, 주석 정리
  • 태그 전체 삭제 기능 추가
  • 자잘한 오류 고침
  • response로 가져오는 내용으로 카테고리 배열 수정

✅ PR Point

  • 지저분하다고 생각되는 코드들 삭제하고, 반복되는 건 함수해서 정리했습니다
  • 서버통신이 많은 부분인데 계속 서버 통신하는 거 안 좋아보여서 처음에만 ssr로 가져오고, 다음부터는 swr를 이용해서 가져왔습니다
  • 자잘한 오류들 수정했습니다 (글자색 유지라든가.. 배너 아이콘 선택후 필터링 돌렸는데 태그 해제하고 그 아이콘에 맞는 리스트가 아니라 전체 리스트 불러와진다든가)
  • 태그 전체 삭제할 수 있는 버튼 기능 추가했습니다

😡 Trouble Shooting

  • useSwr만 사용해서 처음에 가져오는 ssr 통신도 처리하고 싶은데.. 이거 자꾸 오류나고 뭐 안되고 해서.. 아는 사람 방법 추천 부탁ㅜㅅㅜ
  • 이게 통신하지 말고 캐싱해놓고 쓰는 거때문에 useSwr 쓰는건데, 네트워크 확인했을 땐 적절하게 사용되고 있는 것 같은데 확신이 안 들어서 이부분 확인 부탁해욥
  • 지저분한 코드 정리했지만 추가적으로 줄일 수 있거나 효율적인 방향있다면 추천 부탁

👀 스크린샷 / GIF / 링크

리팩토링이후

@say-young516 say-young516 added 🧸 ViewProduct ❕Review 앱잼 이후 리뷰 필요 labels Nov 29, 2022
@say-young516 say-young516 self-assigned this Nov 29, 2022
@netlify
Copy link

netlify bot commented Nov 29, 2022

Deploy Preview for team-nori ready!

Name Link
🔨 Latest commit 8bc054a
🔍 Latest deploy log https://app.netlify.com/sites/team-nori/deploys/639381090d9987000a13c2d7
😎 Deploy Preview https://deploy-preview-164--team-nori.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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
❕Review 앱잼 이후 리뷰 필요 🧸 ViewProduct
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ ViewProduct ] 코드 리팩토링
1 participant