-
Notifications
You must be signed in to change notification settings - Fork 0
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
[INQUIRY] 문의하기 페이지 UI #126
Conversation
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.
고생하셨습니다! 코드 수정 범위가 크지 않아서 보기 편했습니다~! 아래의 부분 추가적으로 수정 부탁드립니다!
- [INQUIRY] 문의하기 페이지 #99 이슈 명칭은 [POST]로 되어 있네요. 통일성을 위해 명칭 수정해주세요!
- 설정의
Service
파일에서 경로가 잘못 되어 있는 것 같습니다. 수정 부탁드립니다! button 타입도 button으로 수정 부탁드려요!
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.
PR크기 적당 👍
수고하셨습니다. 수정하면 좋을 부분들 적어두겠습니다!
|
[#99] request
문의를 작성하는 페이지의 UI를 구현하였습니다.
클래스명이 난잡한데 리팩토링을 진행하며 개선해나가겠습니다.
ToggleButton을 만들었는데 픽셀이 살짝 어긋나는 것처럼 보입니다. 그쪽 css 부분 봐주시면 감사하겠습니다.
Please check if the PR fulfills these requirements
develop
branch, not themain
branchyarn lint
Screenshot