[2단계 - 장바구니 미션] 지그(송지은) 미션 제출합니다. #42
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
HyeonaKwon
approved these changes
Jun 2, 2021
| TABLET: '768px', | ||
| }; | ||
|
|
||
| export default MEDIA_BREAK_POINT; |
There was a problem hiding this comment.
App.styles.ts 에 미디어 쿼리에 관련된 상수들이 있던데, 여기로 옮겨보는건 어때요?
학습로그 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in 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.
안녕하세요! 장바구니 2단계 미션 제출합니다.
늦어서 죄송합니다. 🙇♀️ 🙇♀️ 🙇♀️
🐑 Demo
✅ Todos
👾 Comments
useAppDispatch를 사용해보고 싶었으나, middleware와 함께 사용하는 함수 로직에서는 어째서인지 dependency cycle과 타입 에러가 계속 발생하여 일단 middleware를 사용하지 않는 곳에서만useAppDispatch를 붙여보았습니다... 현재 진행 중인 지하철 노선도 미션에서는 정상적으로 코드가 붙는데 여기서는 왜 그런지 며칠 째 이유를 찾지 못해 일단 사용 가능한 선까지만 사용해 보았습니다!units에 있는 컴포넌트들 중심으로 스냅샷 테스트만 적용해 보았습니다. 현재 지하철 미션에서 RTL을 이용한 E2E 테스트를 작성 중이라 조금 더 공부하고 이후에 살을 더 붙여보도록 하겠습니다.media query를 여러 군데 이용해본 것은 처음인데, 조심스레 평가 부탁드리겠습니다...!더 많은 것들을 해보고 싶었으나... 미션들의 압박으로 이 정도 해보았습니다.
기존 2단계 구현사항이었던 백엔드 api 적용은 1단계에 푸쉬 후 머지되었습니다! 🤓
감사합니다!