2021-pick-git icon indicating copy to clipboard operation
2021-pick-git copied to clipboard

[제안] api/profiles/me API를 제거한다.

Open xlffm3 opened this issue 3 years ago • 3 comments

작업 내용

  • 타인의 프로필을 갈 경우 api/profiles/{username}인데 내 프로필의 경우 api/profiles/me로 갑니다.
  • 기획 초기에는 내가 내 프로필 갔을 때 뷰를 다르게 보여줘야한다는 논의로 인해 위와 같이 API를 분리하게 된 것으로 기억하는데요.
  • 해당 API가 정말로 필요한지에 대해 다시 생각해볼 여지가 있다고 생각합니다.

주의사항

xlffm3 avatar Aug 12 '21 04:08 xlffm3

확실히 username으로 통일하면 좋을 것 같긴한다. 현재 브라우저가 가지고있는 내정보(username, accessToken)이 유효한지 확인할 수 있는 api가 따로 확보된 후에 없애면 좋을 것 같네요! localStorage에 토큰이 유지되고 있는 상태에서 해당 정보가 유효한지 매 진입시마다 확인할 필요가 있습니다

Tanney-102 avatar Aug 12 '21 07:08 Tanney-102

처음부터 주장했던 안건이네요, 통일성이 필요하며 통일 하더라도 동일한 기능을 충분히 구현이 가능하게 제거에 찬성합니다.

bperhaps avatar Aug 12 '21 07:08 bperhaps

굳이 두 API로 나눌 필요가 있을까 생각이 들긴 해요. 저도 제거에 찬성합니다!

da-nyee avatar Aug 12 '21 07:08 da-nyee