-
Notifications
You must be signed in to change notification settings - Fork 7
[조수연] Payment 미션 Step2 (한번 더 리팩토링!...ing) #5
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
Open
suyeon1218
wants to merge
307
commits into
fe-clean-code-study:suyeon1218
Choose a base branch
from
suyeon1218:main
base: suyeon1218
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or 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
- 함수 분리하기
- useState와 useEffect없이 Button렌더링하기 - payments로 사용하던 경로를 payment로 변경하기 - 하위 컴포넌트 이름 미리 작성해두기
- PayCardInformation -> CardSelction으로 이름 변경 - TitleText 컴포넌트 분리 - 경로 별칭 적용 - import-order 적용
- clickable props를 다른 props를 통해서 유추하기 - 보안된 카드 번호들 변수명 변경하기
- onClose, onOpen 의 용도를 이벤트 용도로 변경하기 - 외부에서 초기값과 이벤트 받을 수 있도록하기 - useClosure 를 Provider 내부에서 호출하기
This reverts commit 66d4dea.
- green500 컬러값 수정하기 - label 값 사용하기
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.
구현하면서 어려웠던 점
컴포넌트 구조 변경
widgets,features의 역할이 특히 헷갈려서 파일을 여기저기로 옮겨보는 시도를 한다고 도중에 시간을 많이 잡아 먹었습니다. 조금 더 자세히 공부할 필요가 있다고 생각합니다... ㅠㅠPrimitive UI
Component.Root와 같은 이름은 ark ui의 네이밍에서 따왔습니다.PropsWithChildren과CSSProperties를 모두 일일이 작성했는데ComponentProps하나면 정리가 다 돼서 편리했습니다.궁금한 점 & 리뷰받고 싶은 점
FSD 아키텍쳐
이건 제가 공부를 더 해야하는 부분이지만... entities 에선 도메인 데이터를 담당하는데, 도메인 데이터를 어떻게 분리하는지 감이 잘 안와요. 저희 프로젝트에선 도메인 데이터가
CreditCard만 있는 게 맞을까요?features폴더는 사용자의 비즈니스 시나리오를 담당합니다! 예시로 댓글이 있다면 댓글 쓰기, 댓글 삭제 등이 이에 해당한다고 해요. 저희 프로젝트에선 카드 등록 (추가) / 카드 관리 (삭제, 편집) 으로 나눠봤는데... 이 시나리오로 나누는 게 맞는지 궁금합니다. ㅠㅠ