-
Notifications
You must be signed in to change notification settings - Fork 2
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
[FEATURE] 공통 Modal 구현 및 적용, 홈 & 소개 탭 UI 수정사항 반영 #164
Conversation
|
Deploying sopt-admin with Cloudflare Pages
|
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.
고생하셨습니다 ~~!!
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.
고생하셨습니다~~
홈 탭 UI 변경사항 반영
주용오빠가 작업한 메인버튼 컴포를 아예 날려버리기 미안해서 일단 주석처리만 해놓았음
필요없는 코드의 주석은 레거시가 될 수 있으니 미안해 하지 말고 제거해 주세요~~
✨ 구현 기능 명세
✅ PR Point
변경사항이 매우 많아서 commit 단위로 읽는게 편하실거예요!
혹은 그냥 단순 뷰 수정 작업이었기 때문에 풀받으셔서 뷰만 확인해보셔도 될겁니다!
헤더
모달 추가 (공통 컴포로)소속
필드는 필수 아니므로 required 및 에러처리 제거급하게 한 작업이라 코드퀄리티는 보장하지 못하지만...🥲
그래도 공통 분리 해두었으니 추후 수정 작업은 비교적 쉬울 것 같아요
성공 토스트 메시지
추가해야 함✅ screenshot
영상엔 없지만 지원하기탭 헤더 모달 누락되어있길래 그것도 추가했습니다!
2024-11-29.10.32.26.mov