Skip to content

Conversation

@404yonara
Copy link
Contributor

jwt decode로 memberId를 가져오는 함수를 만들고,
challenges/my-challenge페이지를 [memberId]/challenge로 통합하였습니다.

url도 가독성 좋게
member/[memberId]/challenges로 변경하려고 했지만,
기존의 member라는 단어를 챌린지에 등록된 멤버들이란 의미에서 사용했던것같은데,
해빗페이에 가입한 회원또한 member라는 용어를 쓰면 혼동이 올 것 같아서, 이건 이야기 해보면좋을듯 싶습니다.

우선 url은 기존대로 [userId]/challenge로 하였습니다.

@404yonara 404yonara self-assigned this Jan 8, 2025
@404yonara 404yonara merged commit 95f9c90 into main Jan 8, 2025
1 check passed
@404yonara 404yonara deleted the fix/jwtDecode#161 branch January 8, 2025 20:32
Copy link
Contributor

@Han-Joon-Hyeok Han-Joon-Hyeok left a comment

Choose a reason for hiding this comment

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

우선 url은 기존대로 [userId]/challenge로 하였습니다.

members/[userId]/challenges로 변경하는 건 어떨까요? members를 전체 사용자라고 정의하고, 챌린지 참여자는 participants로 정의하면 좋을 것 같습니다.


그리고 url 이름 지으실 때 아래의 링크 참고하시면 도움될 것 같습니다.

리소스는 복수형으로 표현한다.

// good
www.example.com/reviews
www.example.com/reviews/${reviewId}

// bad
www.example.com/review
www.example.com/review/${reviewId}

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