Open
Conversation
namdaeun
requested review from
Bowoon1216,
cindy-chaewon,
namdaeun and
rtttr1
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
해당 이슈 번호
closed #351
체크리스트
💎 PR Point
주용 부분
파일 추가 API 연결하였습니다.
drive페이지에서changeFile핸들러 구현 시에 여러 파일을 받을 수 있도록 한 후 이를 모두presignedUrl로 변환하여mutate를 수행하는 과정을 거쳤습니다.그리고 폴더를 클릭할 때에는 뷰는 동일하되 해당 드라이브 페이지에서 해당 폴더 안에 있는 파일들을 보여주어야 하는 요구사항이 있었습니다. 이를
url params로 구현하기 위해use-query-params라이브러리를 도입하였습니다. 이는url의 파라미터 값을 상태로서 쉽게 사용할 수 있도록 해주는 라이브러리입니다. 즉 특정 폴더를 클릭하였을 때 해당 상태 setter를 호출하여url의query param을 변경하여 해당 폴더 안으로 이동하는 식으로 구현하였어요 !드라이브 페이지에서 만약
folderId라는param값이 존재한다면api호출 시에도 해당 아이디를 보내 해당 폴더 안의 파일/폴더를 설정할 수 있게 됩니다.