번쩍 홈/전체/상세 #984
번쩍 홈/전체/상세 #984
Conversation
|
| paths['/user/v2/meeting']['get']['responses']['200']['content']['application/json;charset=UTF-8']['meetings'][number], | ||
| 'isCoLeader' | ||
| > & { isCoLeader?: boolean }; | ||
| meetingData: MeetingListOfFilterResponse['meetings'][0]; |
There was a problem hiding this comment.
배열의 정확히 0번째 타입이 필요했던 것일까요?? 그게 아니라면 number 로 적는 것이 일반적이라 number 로 수정해주시면 감사하겠습니다! (나중에 0번째 타입이 필요한 것이라고 헷갈릴 수 있어서요!!)
| @@ -0,0 +1,29 @@ | |||
| import { getLightningById, GetLightningByIdResponse, getLightningList } from '@api/lightning'; | |||
| key: '#환영 태그', | ||
| Title: () => <STitle>#환영 태그</STitle>, | ||
| Content: () => { | ||
| console.log(detailData.welcomeMessageTypes); |
| return ( | ||
| <STarget> | ||
| {detailData?.welcomeMessageTypes.map(tag => ( | ||
| <Chip key={tag} style={{ boxShadow: 'none' }} active> |
There was a problem hiding this comment.
여기 혹시 tag 가 고유한 값이라는 보장이 있을까요??
There was a problem hiding this comment.
네..! 정책상 중복된 값이 들어가지 않습니다!
| import { GetLightningByIdResponse } from '@api/lightning'; | ||
| import { IconLocation } from '@sopt-makers/icons'; | ||
|
|
||
| export const MeetingDetailList = (detailData: GetMeetingResponse) => [ |
There was a problem hiding this comment.
컴포넌트 양이 꽤 되는데, meetingDetailList 가 필요할 때에도 전체 페이지가 리렌더링 되고, LightningDetailList 가 필요할 때에도 전체 페이지가 리렌더링되어서 성능 상의 비효율이 발생할 수 있는 지점인 것 같아요!! 두 개의 컴포넌트가 같이 있을 필요가 없으니 둘을 서로 다른 컴포넌트로 분리하면 어떨까요??
🚩 관련 이슈
📋 작업 내용
📌 PR Point
#976 #976 PR을 머지시켰는데,
PR에 적혀있던 논의사항에 대한 답변 부탁드립니다 !
📸 스크린샷