[FE] 3주차 운영 피드백 반영 - 1(#717) #721
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| @@ -57,7 +57,7 @@ const MyReviewee = ({ roomInfo }: MyRevieweeProps) => { | |||
| return <p>코드리뷰를 하지 않았어요</p>; | |||
| } | |||
|
|
|||
| // TODO: 방 끝나도 계속 작성 가능 | |||
| // TODO: 방이 끝났을 때 피드백 렌더링 정하기 | |||
There was a problem hiding this comment.
내일 QA 해보고 정해질 것 같아요! 그래서 지금 PR에서는 머지가 되고 내일 QA 이후에는 사라지지 않을까 생각합니다~!~!
추가적으로 주석 개수는 3개로 이전과 동일합니다😁
| const [selectedFeedback, setSelectedFeedback] = useState<number | undefined>(() => { | ||
| const savedFeedback = sessionStorage.getItem("selectedFeedback"); | ||
| return savedFeedback ? parseInt(savedFeedback, 10) : undefined; | ||
| }); |
There was a problem hiding this comment.
컴포넌트가 아닌 훅으로 포함시키는게 가독성 측면에서 더 좋은거 같네요 👍
There was a problem hiding this comment.
원래 쓰던 훅이 있어서 바로 가져다 리팩토링 했습니다ㅎㅎ 굳굳
| @@ -13,14 +13,8 @@ const CallbackPage = () => { | |||
| const code = params.get("code"); | |||
|
|
|||
| if (code) { | |||
| postLoginMutation.mutate(code); | |||
| postLoginMutation.mutate(code, { onSuccess: () => navigate("/", { replace: true }) }); | |||
There was a problem hiding this comment.
저번에 언급한 로그인 문제를 해결했네요!
확인해보니 로딩 게이지가 채워지지 않아도 로그인이 되는거 확인했어요~
There was a problem hiding this comment.
setTimeout 2초로 두었을 때 3G 환경에서 로그인이 성공하지 않았는데 메인 페이지로 넘어가 로그인이 안 된 것처럼 화면이 보여지는 오류가 있었습니다!
No throttling 환경에서는 2초 전에 성공해서 UX가 향상되고, 3G 환경에서는 오류를 해결할 수 있어서 이 코드가 정답인 것 같습니다😂
| <dd>{profileData.receivedReviewCount}개</dd> | ||
| </dl> | ||
| <dl> | ||
| <dt>피드백 받은 개수</dt> | ||
| <dt>받은 피드백</dt> |
There was a problem hiding this comment.
리뷰는 작성한 것 받은 것 2가지 보여주는데, 피드백은 받은 것에 대해서만 보여주니까 어색한 것 같아요. 작성한 피드백을 보여주는 건 어떤 것 같나요~?
만약 이걸 추가한다면 평점평균은 키워드의 상단에 나타나도록 해서 현재 디자인을 유지하면 좋을 것 같아용 :)
There was a problem hiding this comment.
흐음.. 작성한 피드백 개수를 서버에서 넘겨주지 않고 있어요🤔
저도 작성한 피드백도 같이 보여주면 좋을 것 같다고 생각해서 백엔드한테 한 번 말씀드려 보겠습니다~!!
| <Button size="small" onClick={rest.onCancel} outline={true}> | ||
| {cancelButtonText} | ||
| </Button> | ||
| <Button size="small" onClick={rest.onConfirm} className="confirm-button"> | ||
| <Button size="small" onClick={rest.onConfirm}> |
There was a problem hiding this comment.
저번에 버튼 컴포넌트를 재사용성있게 수정한 덕분에 더 간결해지고 좋은 것 같아요 👍
There was a problem hiding this comment.
원래 쓰고 있던 곳이 없는 줄 알았는데 className 까지 만들어가며 썼더라구요ㅋㅅㅅㅋㅅㅋ 바로 리팩토링 했습니다😁
📓 스토리북 링크
바로가기
📌 관련 이슈
✨ PR 세부 내용
🎯 1. 기능
받은 피드백,쓴 피드백과선택한 방을 세션 스토리지로 관리📝 2. 수정
프로필 페이지 이름 아래 잘리는 오류 해결
피드백 모아보기에서 textarea 클릭 막기 (readonly -> disabled)
🤔 3. 리팩토링
type > reviewer.ts 의 reviewrInfo가 있는데 reviewee한테도 동일하게 쓰이는 타입이어서 reviewInfo로 변경
Button 컴포넌트 outline 속성이 추가됨에 따라 모달 버튼 리팩토링
상세 페이지 각 섹션별 디자인 통일(gap, 폰트 크기 등)
roomInfo에서 ContentLineBreak 모두 넣기, styled-components 안에 span, div 태그 제거
🎨 4. 디자인
✅ 5. 피드백 반영
로그인 완료 됐을 때 바로 메인 페이지로 리다이렉트
피드백 모아보기 페이지에서 피드백 수정 바로 할 수 있게 변경
