-
Notifications
You must be signed in to change notification settings - Fork 4
✨ Feat: 가게 정보/등록페이지 초안 #68
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
Conversation
|
@BaeZzi813 is attempting to deploy a commit to the projects Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
sohyun0
left a comment
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.
고생하셨습니다! 코멘트 확인해주세요!
| <div> | ||
| <span>상세 주소</span> | ||
| <span className='ml-0.5 text-red-500'>*</span> | ||
| </div> | ||
| <Input | ||
| value={formData.address2} | ||
| onChange={e => handleChange('address2', e.target.value)} | ||
| placeholder='입력' | ||
| /> | ||
| </div> |
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.
dropdown 은 label 이 없지만 기본 input 은 label과 requiredMark 옵션이 있기에 활용해도 좋을것 같습니다 (
이외 다른 인풋도 동일하게 활용한다면 코드 작성량이 줄어들것 같습니다!
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.
작업 초반에 label 부분 font-size 가 달라서 저렇게 작업했는데 지금 다시 해보니 잘되네요 😂 다시 수정했습니다.
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.
아 저도 그부분 font-size 가 달랐어서 제가 저번에 수정하고 PR 올렸습니다!
| <textarea | ||
| value={formData.description} | ||
| placeholder='입력' | ||
| className='h-[153px] resize-none rounded-md border border-gray-300 px-5 py-4' |
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.
base-input 클래스네임을 사용하면 기존 input 스타일을 동일하게 가져갈 수 있습니다:)
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.
네 base-input 적용하고 나머지 heigt 만 추가 했습니다!
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.
height 도 rows로 조절할 수 있으니 참고해보시면 좋을것 같습니다!
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.
페이지 테스트하다 발견한 사항인데 prev , next 버튼을 눌렀을때도 페이지 이동이 되어야할것 같은데 현재는 숫자만 넘어가더라구요
7에서next 버튼 클릭하면 8로 이동후 숫자가 변해야하지 않을까 해서 제안드려봅니다!
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.
해당 부분 이전 그룹 / 이후 그룹 설정 해놓았습니다!
| Register.getLayout = page => ( | ||
| <Wrapper> | ||
| <Header /> | ||
| <main>{page}</main> | ||
| </Wrapper> | ||
| ); |
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.
등록하는 부분에 footer 페이지가 피그마 상에서 존재하지 않지만 로그인, 회원가입 페이지 외에는 전부 있어도 괜찮지 않을까? 하는 개인적인 생각은 있습니다! 이건 팀미팅때 맞춰봐도 좋을것 같습니다!
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.

📝 작업 개요 (필수)
가게 정보 / 등록 페이지 초안입니다.
현재 가게 정보 POST 와 GET 부분에서 image 변환이 완벽하지 않아서 해결을 하고자 하는데 서버 자체에서 내가게 1개 이상 등록시 409 에러 반환으로 막혀있고, 내 가게 DELETE 기능도 없기 때문에 내 가게 편집하기 까지 구현하면서 수정할 예정이라, 미완성 초안입니다.
팀미팅시 공유하였던 Pagination 반응형 페이지개수 / authProvider 리로드시 재인증 문제를 먼저 올려드려야 작업하시기 편하실 것 같아 참고 부탁드립니다.
✨ 작업 내용 (필수)
📸 스크린샷
🧐 해결해야 하는 문제
🤔 리뷰어 확인 필요 사항
🔗 관련 이슈
🛠️ 후속 작업
✅ 체크리스트 (필수)