-
Notifications
You must be signed in to change notification settings - Fork 51
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
[홍재원] week19 #557
The head ref may contain hidden characters: "part3-\uD64D\uC7AC\uC6D0-week19"
[홍재원] week19 #557
Conversation
…ithub-actions [Fix] delete merged branch github action
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 중인데 질문 주신거 먼저 코멘트 드리고 마저 보겠습니다!
재원님 파트3 고생 많으셨습니다! 대공사 하시느라 엄청 고생하셨겠군요.. 👍 가끔 useQuery가 제대로 cache를 저장하지 못하는 모습이 보입니다.
공유 페이지인 sharedPage의 url을 어떻게 짤지 고민입니다. 이걸 구현하기 위해 shared/{userId}/{folderId} 이렇게 url을 설정할지,
response로 401 status를 받을 경우, refresh 토큰으로 accessToken을 재발급받도록 axios interceptors 코드를 짰지만 제대로 동작하지 않습니다. 왜 이런건지 모르겠습니다....
|
요구사항
후딱 수정해보겠습니다, 늘 죄송하고 감사합니다!!
기본
주요 변경사항
고칠 점
모달
스크린샷
랜딩
로그인 직후 /folders 페이지로 이동.
폴더 추가 버튼 누르면 바로 추가되고, folderList도 새로고침없이 자동으로 업데이트됨
이름 변경도 자동 데이터패칭 되고 잘 업데이트 됨.
링크 삭제를 누르면 삭제됨
멘토에게
가끔 useQuery가 제대로 cache를 저장하지 못하는 모습이 보입니다.
/folders/1 페이지에서 /folders/2 페이지로 넘어갈 때 어떨 때엔 folders-2 캐시가 잘 저장되고, 어떨땐 아예 저장이 안되고 랜덤으로 작동합니다... 근데 왜 이런건지를 모르겠습니다.
공유 페이지인 sharedPage의 url을 어떻게 짤지 고민입니다.
sharedPage 기능: 외부 유저에게서 공유받은 페이지로, 외부 유저의 id와 외부유저가 공유해준 폴더 아이디, 그 폴더의 card list들이 보이는 페이지입니다.
https://www.figma.com/file/KjzGnP6LY48fN0NCFzp0iQ/Weekly-Mission?type=design&node-id=15555-64804&mode=design&t=zp3V6FgiCyprFnYr-0
이걸 구현하기 위해 shared/{userId}/{folderId} 이렇게 url을 설정할지,
아니면 shared?userId={userId}&folderId={folderId} 이렇게 쿼리문으로 url을 설정해서 처리할지 고민입니다!! 어떻게 해볼까요??
response로 401 status를 받을 경우, refresh 토큰으로 accessToken을 재발급받도록 axios interceptors 코드를 짰지만 제대로 동작하지 않습니다. 왜 이런건지 모르겠습니다....
셀프 코드 리뷰를 통해 질문 이어가겠습니다.