-
Notifications
You must be signed in to change notification settings - Fork 6
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/server] 모달 컴포넌트 공통화 #158
Conversation
infra/Chart.lock
Outdated
dependencies: | ||
- name: ingress-nginx | ||
repository: https://kubernetes.github.io/ingress-nginx | ||
version: 4.10.1 | ||
digest: sha256:d25ff4f8cab798c10c0cf44b9cac931287596320e9645bd294dc9eaaa5717bd9 | ||
generated: "2024-06-05T19:52:15.651144+09:00" |
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.
이 파일 삭제하고 .gitignore에 등록해주실 수 있을까요?
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.
파일 삭제하고 .gitignore에도 추가했습니다~!
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.
타입스크립트 처음 써봐서 잘 모르는데 types에 style관련 코드도 같이 작성하는 이유가 궁금합니다
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.
아마 제가 스타일 폴더 만들어서 스타일 파일 분리해 놓는 것을 깜빡한것같습니다. 🤦♀️
스타일 폴더 만들어서 넣어 놓겠습니다~!
#️⃣연관된 이슈
📝작업 내용
다른 컴포넌트에서 모달을 사용하고 싶다면 아래와 같이 사용하시면 됩니다.
###스크린샷 (선택)
💬리뷰 요구사항(선택)
제가 모달 컴포넌트 공통화 시킨다고 예원님 코드를 수정 했습니다. 혹시 충돌 해결 하시다가 문제생기시면 알려주세용~!