✨ 프로필 유저정보 Skeleton UI

[Legitgoons]

작업 이유

• Loading 중 출력될 프로필 유저정보 UI

작업 사항

• 프로필 유저정보 Skeleton 구현

리뷰어가 중점적으로 확인해야 하는 부분

• 가능하다면 직접 브랜치를 pull해서 response에 delay를 추가해서 봐주시면 감사드리겠습니다.
• 기획대로 알맞게 화면이 출력되나요?

발견한 이슈

• 현재 scss에서 Wrapper - Container - Box의 단계가 확립되어 있지 않습니다. 이를 컨벤션으로 정해놓는 것이 코드 작성 시 도움이 될 것 같습니다.
• ✨ User API 닉네임 / 이름 분리 #89 닉네임 / 이름 분리 필요

[aws-amplify-ap-northeast-2]

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-86.d37mn03xh3qyyz.amplifyapp.com

[BangDori]

확인하였습니다~

근데, 현재 스켈레톤 UI는 잘 동작하고 있는데, 위와 같이 해당 프로필 사용자의 닉네임이 표시되고 있지 않은 것 같아요! 이 부분 확인 부탁드릴게요!

---

현재 scss에서 Wrapper - Container - Box의 단계가 확립되어 있지 않습니다. 이를 컨벤션으로 정해놓는 것이 코드 작성 시 도움이 될 것 같습니다.

저도 이 부분에 대해 동일한 의견입니다! 근데, 이미 어느정도 작업이 진행되어 있는 상황이여서 오늘 리뷰 회의 끝나고 이야기 할까요?

[Legitgoons]

지금 해당 페이지 관련 문제를 정리하면 다음과 같습니다.

1. 현재 User API 닉네임과 이름이 별개로 되어 있지않고 name이라는 하나의 키워드만 있습니다. 전체적인 수정이 필요해 보입니다.
2. 이름을 header에 미리 띄워주기 위해서는, FeedMainPage에서 상태를 전달해줘야합니다. 그런데 FeedMainPage에서는 이미 profileImage와 name을 가지고 있습니다. 그렇다면 굳이 이름만 header에 띄워주지 말고, profileImage와 이름, 닉네임을 loading 단계에서 출력하는 것도 고려해봐야 할 것입니다.

따라서 다음 의견을 제시드립니다.

1. 현재의 PR은 병합한다.
2. User API의 닉네임과 이름 분리 문제는 이슈로 등록하고 별개의 PR로 처리한다.
3. loading 단계에서 어느 정도까지 정보를 출력할지는 PM 및 디자인 팀과 협의한다.

[BangDori]

네 좋습니다. 그럼 우선 PR 병합하고, 이슈로 등록한 이후에 디자인 팀과 협의해서 이슈 처리하면 될 것 같습니다~