대시보드 페이지내에 진행 중인 미션 리스트와 내가 제출한 솔루션 구현 (issue #272) #299
Conversation
| return data; | ||
| }; | ||
| // TODO 실제 API 연결 아직 안했습니다. | ||
| export const getMissionInProgress = async () => { |
There was a problem hiding this comment.
이전 기획의 미션 현황 페이지에서 사용 중이던 '진행 중 미션', '완료한 미션' 관련 코드가 더 이상 쓰이지 않을거라 생각해서 삭제했는데 (PR) 대시보드에서 쓰이고 있었군요..😢
There was a problem hiding this comment.
전에 기획단계에서 있었던 진행 중 미션과 완료한 미션이 서로 네이밍이 겹치더라고요🥲
그래서 이번엔 아예 제가 새롭게 선언해보았습니다!
| export const getSubmittedSolution = async (): Promise<SubmittedSolution[]> => { | ||
| return SubmittedSolutions; | ||
| }; |
There was a problem hiding this comment.
반환 데이터가 타입인 이유가 있을까요~?
There was a problem hiding this comment.
앗 타입명이랑 목데이터랑 이름이 비슷해서 착오가 있었던거 같습니다..!
현재 리턴해주는 데이터는 제가 mocks폴더 내에 선언해놓은 목데이터입니다! 🙇
후에 실제 API랑 연동해서 헷갈리는일 없도록 하겠습니다~!
There was a problem hiding this comment.
아하 목데이터였군요! 알려주셔서 감사합니다 ㅎㅎ
| <S.MainText>진행 중인 미션이 없어요</S.MainText> | ||
| <S.SubText>새로운 미션을 찾으러 가볼까요?</S.SubText> |
There was a problem hiding this comment.
솔루션과 진행 중 미션이 없을 때의 메세지가 서로 다른데요, 재사용성을 위해 위 문구를 props로 받거나 타입을 받아 내부에서 분기처리를 하면 어떨까요?
There was a problem hiding this comment.
오호 제가 놓쳤던 부분이에요!
짚어주셔서 감사합니다 🙇 반영하도록 하겠습니다!
| <Mission.InfoWrapper> | ||
| <Mission.Title>{mission.title}</Mission.Title> | ||
| <Mission.Summary>{mission.summary}</Mission.Summary> | ||
| <Mission.HashTag hashtagList={mission.hashtag} /> |
There was a problem hiding this comment.
camelCase를 적용해서 hashtag를 hashTag로 작성하는 것은 어떨까요~? HashTag 컴포넌트는 Tag가 대문자로 되어있어서요 ㅎㅎ
There was a problem hiding this comment.
짚어주셔서 감사합니다~! 반영하도록 하겠습니다 🙇
| width: 100%; | ||
| `; | ||
|
|
||
| export const MissionItemContainer = styled.article` |
There was a problem hiding this comment.
오 article이라는 태그도 있군요! 새롭게 배워갑니다 👍
| `; | ||
|
|
||
| export const MainText = styled.span` | ||
| font-size: 2rem; |
There was a problem hiding this comment.
폰트 사이즈가 1.6rem이 되어야 할 것 같아요~!
There was a problem hiding this comment.
짚어주셔서 감사합니다! 수정해서 반영하였습니다! 🙇
구현 요약
연관 이슈
close #272
참고
코드 리뷰에
RCA 룰을 적용할 시 참고해주세요.