[FOLLOW] 친구 목록 및 친구 프로필 페이지 구현 #96
Conversation
chaeseungyun
requested review from
ChoiWonBeen,
jaewoogwak,
hyejun0228 and
KimKyungYun
as code owners
| import { ReactComponent as Empty } from 'assets/svg/follow/no-friend.svg'; | ||
| import style from './FailToSearch.module.scss'; | ||
|
|
||
| export default function EmptyFriend() { |
There was a problem hiding this comment.
EmptyFriend 컴포넌트와 FailToSearch 컴포넌트의 구조가 비슷한데 AuthDetail 컴포넌트처럼 인자로만 전달하는 방식을 사용하면 어떨까요..?
| <div className={style.user__info}> | ||
| <div> | ||
| <span className={cn({ [style['user__info--span']]: true })}>{state.nickname}</span> | ||
| <span>count</span> |
There was a problem hiding this comment.
아직 기능하나가 추가가 안 된상태였습니다.
count는 팔로워가 작성한 리뷰의 총 개수입니다
kimeodml
requested review from
HAEROOL and
haejinyun
and removed request for
jaewoogwak
| align-items: center; | ||
| overflow: scroll; | ||
| gap: 10px; | ||
| width: 1067px; |
There was a problem hiding this comment.
위의 타이틀의 width: 1187과 더불어서 고정 width가 불편한 것 같아요.
space-between을 유지한 채로 100%를 두되, max-width를 지금의 수치로 고정하는 방법은 어떨까요??
| import style from './Follower.module.scss'; | ||
|
|
||
| // 팔로우 요청 후 유저 목록을 다시 받아와 요청중 상태로 변경 | ||
| const useRequestAndUpdate = () => { | ||
| export const useRequestAndUpdate = () => { |
There was a problem hiding this comment.
Mutation이라는 표현이 RequestAndUpdate를 함축한다고 생각하는데 useFriendListMutation.. 비슷한? 표현은 어떨까 의견 내봐요!
| onClick={ | ||
| () => (followedType === 'NONE' && request(account)) | ||
| || (followedType === 'REQUEST_RECEIVE' && requestId && accept(requestId)) | ||
| || (followedType === 'FOLLOWED' && del(account)) | ||
| || (followedType === 'REQUEST_SENT' && requestId && cancel(requestId)) | ||
| } |
There was a problem hiding this comment.
switch문으로 이루어진 handleClick 콜백을 만들면 좀 더 깔끔할 것 같기두요??
아래 111L ~ 조건문들까지 생각하면 onClick Function과 UI로 이루어진 객체의 배열을 만들어 반복문과 객체 접근으로 표현하는건 어떨까 싶어요!
| const useGetDetailReview = (placeId: string, followerId: number) => { | ||
| const { data } = useInfiniteQuery( | ||
| ['detail', placeId], | ||
| ({ pageParam = '' }) => getDetailReview(followerId, placeId, pageParam), |
There was a problem hiding this comment.
여기에 pageParam이 반드시 undefined로 들어가는데 추후 확장을 고려한 부분일까요?
| placeId, name, photos, isCheckerboard, category, | ||
| }: Props) { | ||
| const { isMobile } = useMediaQuery(); | ||
| const { data } = useGetDetailReview(placeId, 361); |
There was a problem hiding this comment.
팔로워가 아닌 상황에 대해 에러가 발생하는데 해당 쿼리의 onError 콜백에 알림과 함께 이전 페이지로 되돌리는 UX라거나 할만한 부분들을 고려해볼 수 있을 것 같아요
chaeseungyun
requested review from
LUNA-KK,
junghaesung79 and
20HyeonsuLee
as code owners
친구 목록 및 친구 프로필 페이지를 구현했습니다.
프로필은 오직 팔로워끼리만 확인할 수 있습니다.
친구 프로필 페이지의 pc 디자인이 확정되지 않아 모바일뷰를 많이 참고했습니다. 추후 수정하겠습니다.
Please check if the PR fulfills these requirements
developbranch, not themainbranchyarn lintScreenshot