Conversation
There was a problem hiding this comment.
build 결과물들은 내 소스 코드가 변하면 함께 변하는 것이기 때문에 git으로 등록할 필요가 없습니다.
이런 내용물들은 .gitignore에 등록해주세요.
| "devDependencies": { | ||
| "typescript": "^5.4.5" |
There was a problem hiding this comment.
typeScript 자체는 결과물에 포함이 안되죠.
dev에 잘 구분해서 적절히 추가두셨네요. 👍
|
|
||
| function AddLinkInput() { | ||
| const AddLinkInput: React.FC = () => { |
There was a problem hiding this comment.
React.FC는 사용하지 않는 편이 좋습니다.
| interface Link { | ||
| created_at: string; | ||
| description: string; | ||
| folder_id: number; | ||
| id: number; | ||
| image_source: string; | ||
| title: string; | ||
| updated_at: number | null; | ||
| url: string; | ||
| [key: string]: any; | ||
| } |
There was a problem hiding this comment.
이와 같은 type들이 다른 파일에서 여럿 보이네요.
필요할때마다 유사하게 타입을 선언하기 보단 하나의 Link 타입을 가지고 재활용해주는 편이 좋습니다.
| link: { | ||
| created_at: string; | ||
| description: string; | ||
| image_source: string; | ||
| title: string; | ||
| url: string; | ||
| }; |
There was a problem hiding this comment.
CardList 컴포넌트의 Link type의 부분적인 형태같네요.
하나의 Link를 가지고 필요한 property만 빼서 새로운 type으로 만들어 사용할 수도 있습니다.
Omit, Partial 등의 Utility type들을 한번 사용해보세요.
| interface Link { | ||
| created_at: string; | ||
| description: string; | ||
| folder_id: number; | ||
| id: number; | ||
| image_source: string; | ||
| title: string; | ||
| updated_at: number | null; | ||
| url: string; | ||
| [key: string]: any; | ||
| } |
There was a problem hiding this comment.
동일한 Link 타입이 또 있네요.
interface의 경우 말씀드렸듯이, 이름이 같으면 알아서 타입이 확장됩니다.
따라서, 어느순간 내 의도와는 다르게 타입이 변해버릴지 몰라서 굉장히 위험합니다.
하나의 타입을 재활용하던가 타입 이름을 Unique하게 만들어주세요.
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게