[홍건우] Payment 미션 Step1 #2
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
| const Modal = ({ children, showModal, onClose }: ModalProps) => { | ||
| const modalLayoutRef = useRef<HTMLDivElement>(null); | ||
|
|
||
| const handleClickModalLayout = useCallback( | ||
| (e: MouseEvent<HTMLDivElement>) => { | ||
| e.currentTarget === e.target && onClose(); | ||
| }, | ||
| [onClose] | ||
| ); | ||
|
|
||
| return ( | ||
| showModal && | ||
| createPortal( | ||
| <ModalLayout ref={modalLayoutRef} onClick={handleClickModalLayout}> | ||
| {children} | ||
| </ModalLayout>, | ||
| document.body | ||
| ) | ||
| ); | ||
| }; |
There was a problem hiding this comment.
onClose 라는 이름에 대해서 조금 고민해볼 필요가 있어요!
여기서 onClose는 �close가 발생했을 때 바깥에 알리는 역할일까요?
아니면 close를 실행해주는 역할을 하는걸까요?
반대로, onClose의 정확한 역할은 무엇이 되어야할까요?
| <CardBox> | ||
| <Card cardInfo={cardInfo} /> | ||
| </CardBox> | ||
| <CardCreateForm cardInfo={cardInfo} setCardInfo={setCardInfo} /> |
There was a problem hiding this comment.
setCardInfo 대신에 onChange 같은걸로 받아오면 어땠을까요!?
2 participants
생각보다 시간이 오래 걸리는 구현이었습니다;;
지금 상태는 카드 등록을 위한 폼까지만 작업을 해두었습니다.
기타 추가적인 기능과 페이지는 step2에서 구현해보고자 합니다.
배포 링크
미션을 수행하면서 어려웠던 점
리뷰 받고 싶은 부분
궁금한 점
내가 어떤 지점에서 재미를 많이 느끼는지, 희열을 많이 느끼는지, 몰입을 많이 하는지