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

Part2 성은지 week7 #308

Open
wants to merge 3 commits into
base: part2-성은지
Choose a base branch
from

Conversation

sung-eunji
Copy link
Collaborator

요구사항

기본

  • 링크 공유 페이지 url path는 ‘/shared’, 폴더 페이지 url path는 ‘/folder’가 되도록 설정했나요?
  • 폴더 페이지에서 상단 네비게이션 바는 스크롤 시 상단에 고정하지 않고 가려지도록 했나요?
  • 상단 네비게이션 바에 프로필 영역의 데이터는 https://bootcamp-api.codeit.kr/docs 에 명세된 “/api/users/1”을 활용했나요?
  • “전체” 폴더를 선택한 경우 “공유”, “이름 변경”, “삭제” 버튼들이 보이지 않지만, 다른 폴더를 선택한 경우에는 버튼들이 보이나요?
  • 폴더 목록에 필요한 데이터는 “/api/users/1/folders”를 활용했나요?
  • “전체” 폴더에 필요한 링크들 데이터는 “/api/users/1/links”를 활용하고, 이외의 폴더에 필요한 링크들 데이터는 “/api/users/1/links?folderId={해당 폴더 ID}”를 활용했나요?
  • Mobile에서 “폴더 추가” 버튼은 최하단에서 101px 떨어져있는 Floating Action Button으로 만들었나요?

심화

  • [x]
  • []

주요 변경사항

스크린샷

image

멘토에게

  • 데이터를 필터링하기가 되지 않습니다. onClick 이벤트에 쓰이는 handler함수에 필터링 된 데이터를 넣는것이 되고있지 않습니다.
  • 모달창을 useState 사용해서 연다는것까지만 알고 어떻게를 잘 모르겠습니다.
  • 컴포넌트 파라미터에 쓰이는 props들을 활용하는법을 모르겠습니다. ㅠ..
  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@sung-eunji sung-eunji added the 미완성 죄송합니다... label Dec 7, 2023
@sung-eunji sung-eunji requested a review from yujo11 December 7, 2023 05:56
@yujo11
Copy link
Collaborator

yujo11 commented Dec 12, 2023

@Nebaisgood 님 같이 구현 진행한 이후로 추가로 작업하신 내용이 있으실까요?

Copy link
Collaborator

@yujo11 yujo11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

추가 구현 하신 후 리뷰 받고 싶으실 때 아래 버튼 눌러서 다시 리뷰요청 주세요 은지님~

image

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