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

[권주현] Week15 #474

Merged

Conversation

kuum97
Copy link
Collaborator

@kuum97 kuum97 commented May 26, 2024

요구사항

기본

  • 링크 공유 페이지의 url path를 ‘/shared/{folderId}’로 변경하고, {folderId}에 해당하는 폴더 데이터가 화면에 보이게 해주세요.
  • 폴더 페이지의 url path를 전체 링크를 보는 경우 ‘/folder’로, 특정 폴더를 보는 경우 ‘/folder/{folderId}’로 변경하고, {folderId}에 해당하는 폴더 데이터가 화면에 보이게 해주세요.
  • https://bootcamp-api.codeit.kr/docs 에서 인증이 필요한(자물쇠 아이콘이 있음) api의 경우 Authorization 리퀘스트 헤더에 “Bearer {accessToken}”을 함께 보내야 합니다.

심화

  • [] 리퀘스트 헤더에 인증 토큰을 첨부할 때 axios interceptors를 활용해 주세요. (axios를 사용하지 않는다면 이와 유사한 기능을 활용해 주세요.)

주요 변경사항

  • 링크카드를 렌더링해주는 페이지의 레이아웃을 만들어 가독성을 개선하고 불필요한 컴포넌트를 제거했습니다.
  • 저번에 구현 못했던 로그인 로직과 비밀번호 가시성 토글을 구현했습니다.
  • 유저, 폴더, 링크에 대한 api로직을 react query를 이용하여 커스텀훅으로 구현했습니다.
  • 로그인시 토큰을 로컬스토리지에 저장하고 폴더 페이지에 진입하면 해당 토큰으로 클라이언트 사이드에서 데이터 요청을 합니다.
  • 각 폴더의 아이디로 동적 라우팅을 구현하였습니다.
  • zustand와 react-hook-form으로 검색바의 로직을 구현했습니다.

멘토에게

  • 액세스토큰을 로컬스토리지에 저장하여서 서버사이드렌더링이 불가능하였는데 원래는 서버측에서 토큰을 인증하고 서버사이드로 유저데이터같은 것들을 요청하는게 맞나요??
  • 동적 라우팅을 구현했는데 레이아웃 이외에 중복되는 컴포넌트들이 있는데 이들이 전역상태관리되고 있다면 동적라우팅을 위한 레이아웃도 만들어야 하나요??
  • 다른 분들의 요청이 없다면 라이브 코드리뷰 부탁드립니다!

@kuum97 kuum97 requested a review from devToram May 26, 2024 09:58
@kuum97 kuum97 self-assigned this May 26, 2024
@kuum97 kuum97 added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label May 26, 2024
@devToram
Copy link
Collaborator

어제 라이브 리뷰로 갈음합니다!
매번 머지 늦어 죄송해요 ㅜ

@devToram devToram merged commit 847bcb9 into codeit-bootcamp-frontend:part3-권주현 May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants