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

REQUEST: useUser.tsx 전반적 수정 요청 #253

Closed
galaxy4276 opened this issue Aug 6, 2021 · 3 comments · Fixed by #254
Closed

REQUEST: useUser.tsx 전반적 수정 요청 #253

galaxy4276 opened this issue Aug 6, 2021 · 3 comments · Fixed by #254
Labels
frontend 프런트 작업 request change 변경 요청 issue

Comments

@galaxy4276
Copy link
Collaborator

  • 백엔드에서 내려주는 값과 다릅니다. ( user 정보 props 를 사용하신 부분이 없어서 개발 중 문제가 없었 던 것으로 판단 됨 )
  • 타입이 제대로 작동하지 않습니다. IUserSuccess... | IUserFail... 부분에 의해 정상적으로 동작하지 않는 듯 합니다.
  • 주석 정보가 너무 많습니다. 코드 시작 이후 주석은 없어도 될듯하고, JSDoc 으로 주신 부분도 너무 과합니다.
  • page/chat/[id].tsx 에서 userId 를 넣어주는 부분에서 정상적으로 추론되어 사용 될 수 있도록 바꿔주세요.

결론적으로, [id].tsx 에서 사용하는데 문제 없게 끔 수정해주시면 좋을 것 같습니다.
파일을 확인해보니 백엔드에서 바꿔주어야할 필요성은 없어보입니다.

@galaxy4276 galaxy4276 added frontend 프런트 작업 request change 변경 요청 issue labels Aug 6, 2021
@bear-bear-bear
Copy link
Owner

타입 수정 완료했습니다 죄송합니다.

user 응답도 통상 서버 응답 구조 스키마를 포함하는 걸로 잘못 알아서 생긴 타입 오류였습니다.

@bear-bear-bear
Copy link
Owner

bear-bear-bear commented Aug 8, 2021

서버에서 유저 정보에 대한 응답이 아래와 같이 오고 있는데,

image

기존 요청드린 기획은 응답으로 온 데이터 내부에 user 정보가 spread 되어 있는 건데, 지금은 user 정보가 한번 wrapping 되어 있는 듯 합니다. 수정해주세요~ (해당 API 요청 시 구조를 제대로 명시하지 않은 제 잘못인 듯 합니다)


// 기대했던 것
기대 {
  isLoggedIn: true,
  id: ...;
  username: ...;
  email: ...;
  skills: ...;
  job: ...;
}
// 현재 응답
현재 {
  isLoggedIn: true,
  user: {
    id: ...;
    username: ...;
    email: ...;
    skills: ...;
    job: ...;
  }
}

@bear-bear-bear
Copy link
Owner

bear-bear-bear commented Aug 8, 2021

image

chat 페이지에 작성하신 코드 중, 저 id가 undefined인것도 위 이유 때문입니다.
기대하신건 user.id 이지만 현재 응답 구조상으론 user.user.id 인 것..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend 프런트 작업 request change 변경 요청 issue
Projects
None yet
2 participants