-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[FIX] config -> constants 폴더 path 변경 #27
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
aeong98
approved these changes
Mar 4, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!!!!!!
(cherry picked from commit f37675e)
(cherry picked from commit f37675e)
* feat: react/no-unknown-property css 추가 * feat: Button 컴포넌트 추가 렌더링 테스트 코드 작성 * feat: Text 컴포넌트 추가 test: 렌더링 테스트 및 속성 별 테스트 코드 추가 * refactor: theme을 as const로 readonly하게 변경 * fix : Checkbox isChecked 필수값으로 변경 * fix : 사용하지 않는 props 제거 * test: 버튼 클릭 테스트 추가 * feat: Select 컴포넌트 개발 * feat: ButtonProps export * feat: Select 로컬 상태값 추가 * feat: test util 추가 * test: Select.test.tsx 추가 * test: Select.test.tsx 아이템 선택후 닫기 테스트 추가 * test: Select.test.tsx 깨지는 테스트코드 수정 * test: Select.test.tsx 깨지는 테스트코드 수정 (cherry picked from commit f37675e) --------- Co-authored-by: aeong98 <sy9815@gmail.com> Co-authored-by: aeong98 <51521314+aeong98@users.noreply.github.com>
* feat: 검색 로직 추가, hook 분리 * feat: 테스트 코드 정리 * feat: 검색 관련 테스트코드 추가 * test: 코드 정리 * test: 상수 uppercase로 변경 * fix: 불필요한 옵셔널 체이닝 수정
* feat: slideItem 추가 * test: 테스트 코드 추가 * feat: 슬라이드 이벤트 시작 상태값 추가 * feat: 주석 추가 * feat: handleSlideLeftEvent 로직 추가 * feat: 코드 정리 * test: mouse,touch 이벤트 추가 * fix: props interface omit * fix: touch event 테스트 깨지는 부분 수정 * fix: touch event handler 잘못 연결되어 있던것 수정 * 모바일 환경에서의 테스트를 위한 커밋 * Revert "모바일 환경에서의 테스트를 위한 커밋" This reverts commit 345d321. * fix: theme에서 바로 상수 적용 * fix: wrapperWidth 계산 후에 option 렌더링 * fix: width 초기값 세팅
Coverage report
Test suite run success40 tests passing in 10 suites. Report generated by 🧪jest coverage report action from c6011a4 |
ww8007
added a commit
that referenced
this pull request
Mar 5, 2023
[FIX] config -> constants 폴더 path 변경
ww8007
added a commit
that referenced
this pull request
Mar 10, 2023
[FIX] config -> constants 폴더 path 변경
ww8007
added a commit
that referenced
this pull request
Mar 10, 2023
[FIX] config -> constants 폴더 path 변경
ww8007
added a commit
that referenced
this pull request
Mar 10, 2023
[FIX] config -> constants 폴더 path 변경
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌 개요 (필수)
config -> navigate path 변경
🔨 작업 사항 (필수)
config의 경우 라이브러리에 대한 config에 대한 내용과 혼동이 있을 수 있으니
이를 constants로 변경
⚡️ 관심 리뷰 (선택)
🌱 연관 내용 (선택)