Skip to content

Conversation

@usageness
Copy link

안녕하세요 오스틴!
step3로 다시 만나게 되어 반갑습니다 😊

이번 step은 요구사항 충족과 피드백이 필수가 아닌 것으로 알고 있어요!
그럼에도 꼭 완료하고 싶었지만 개인 일정과 다른 일들이 겹치다보니
타입스크립트 학습도 충분하지 않고.. 만족할 수준으로 미션을 수행해서 제출하지 못했네요 😥

오스틴이 이전에 해주셨던 피드백과 새로 학습하는 공포의 타입스크립트를 바탕으로...
미션이 끝나고 나서도 계속 리팩토링하며 개인적인 학습을 이어가도록 하겠습니다!

오늘도 좋은 하루 되시길 바랄게요! 😄


필수 요구사항

  • JavaScript => TypeScript 마이그레이션
  • 최소 1개 이상의 파일을 TS로 마이그레이션한다.

  • 프로젝트에 타입스크립트를 사용할 수 있도록 세팅하였습니다.
  • 공통적으로 사용되는 common 컴포넌트를 TS로 변환 하였습니다.
  • 전역 컨텍스트를 TS로 변환 하였습니다.
  • 기타 상수를 TS로 변환 하였습니다.

usageness added 26 commits May 28, 2022 16:06
Copy link

@austinpark420 austinpark420 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

안녕하세요 유세지 😄

여러가지 일정도 있고 학습할 공부분량도 많고 많이 힘드시죠 ㅠㅠ
지치지말고 화이팅하시면 나중에 좋은 결과 있을거에요!!
이번 미션은 머지하도록 할게요.

고생많으셨습니다 👍

height: 130px;
background: ${(props) => props.backgroundColor || "#D2D2D2"};
background: ${(props) => props.backgroundColor || COLORS.WHITE_200};

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

상수로 잘 빼주셨네요 👍

@@ -0,0 +1,35 @@
interface card {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

interface, type 첫문자는 보통 대문자로 작성합니다!~

@austinpark420 austinpark420 merged commit 2d19a85 into woowacourse:usageness May 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants