Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/feedback #37

Merged
merged 21 commits into from
Mar 23, 2022
Merged

Feat/feedback #37

merged 21 commits into from
Mar 23, 2022

Conversation

alittlekitten
Copy link
Owner

@alittlekitten alittlekitten commented Mar 23, 2022

  • 12시간제 시계에서 12시가 00시로 표시되는 오류 수정 👉 issue 링크
  • + - 버튼 마우스 커서 pointer로 바꾸기 👉 issue 링크
  • craco 적용을 통한 JSX pragma 제거
  • craco 적용을 통한 절대경로 설정
  • 필요없는 logo.svg파일 삭제
  • padStart를 통한 코드 리팩토링
  • eslint, prettier 적용 및 @typescript-eslint 적용
  • eslint 설정으로부터 발견된 미비한 타입 및 인터페이스 관련 문제 및 오류 해소, any type 제거
  • 재생 버튼과 일시정지 버튼을 조건부 렌더링으로 화면에 띄워서 UI/UX 개선

emotion의 JSX pragma를 babel preset 형태로 변환하기 위해서 craco를 도입했습니다.
문제 없이 동작하는 모습 확인 완료
플러그인 추가 및 no-unused-vars 옵션 적용
타입 선언 및 필요한 props 타입 한번에 가져오기
hooks에서 발생할 수 있는 다양한 타입 예외 상황에 대한 처리 완료 및 any 제거
@를 앞에 붙이는 형태로 설정
이벤트에 필요한 React를 useTimer에서 import
그 외 emotion에서 사용하는 props에 대한 interface 추가
craco 설치, typescript-eslint 관련 요소들 추가, typescript 버전 4.6.2로 업그레이드
@alittlekitten alittlekitten added documentation Improvements or additions to documentation deployment setting to deploy labels Mar 23, 2022
@alittlekitten alittlekitten self-assigned this Mar 23, 2022
@alittlekitten alittlekitten merged commit d401d7e into main Mar 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployment setting to deploy documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant