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

Fix: 폴더 구조 변경 #56

Merged
merged 6 commits into from
Feb 23, 2024
Merged

Conversation

Nahyun-Kang
Copy link
Contributor

@Nahyun-Kang Nahyun-Kang commented Feb 23, 2024

개요

  • 개요는 변경 사항 및 관련 이슈에 대해 간단하게 작성해주세요.
  • 해당 PR에 대한 리뷰어와 라벨을 생성해 주세요.

작업 사항

  • 경로 일괄 수정했습니다!
  1. 리스트 상세 조회 페이지: user/[userId]/list/[listId] -> list/[listId]
  2. 히스토리 페이지: list/[list]/history
  3. 수정 페이지: list/[list]/edit

-제가 작업한 것들 중 잘못된 경로들은 수정해주었습니다.
-라우터 훅 쓰던 버튼들은 멘토님의 조언에 따라 링크태그로 교체했습니다.


참고 사항 (optional)

  • 작업 중 만난 버그, 구현 과정 중 어려움, 고민
  • 해결 방법
  • 해당 라이브러리를 선택한 이유
  • 테스트 계획

관련 이슈 (optional)


스크린샷

image


리뷰어에게


Copy link

vercel bot commented Feb 23, 2024

@Nahyun-Kang is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@seoyoung-min seoyoung-min left a comment

Choose a reason for hiding this comment

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

우왓,,, 번거로운 일인데 맡아 해주셔서 감사합니다ㅠㅠ! 덕분에 편하게 이동할 수 있을 것 같아요!! LGTM👍👍
+) 꺄 나 박제당함...!!ㅋㅋㅋㅋㅋㅋㅋ

Copy link
Contributor

@ParkSohyunee ParkSohyunee left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 나현님~~!! 저도 라우터 훅 사용하는 부분들 링크태그로 다 리팩토링해야겠어요. 🙇‍♀️🥰

@Nahyun-Kang Nahyun-Kang merged commit 1f2da96 into 8-Sprinters:dev Feb 23, 2024
1 of 2 checks passed
@Nahyun-Kang Nahyun-Kang deleted the fix/move-folders branch February 23, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants