-
Notifications
You must be signed in to change notification settings - Fork 4
✨ Feat: 가게 등록 공고 조회 / 무한스크롤 구현 #86
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.
💡 Enable Vercel Agent with $100 free credit for automated AI reviews |
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.
고생하셨습니다! 코멘트 확인해주세요:)
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.
해당부분 export 변경되면서 다른파일에서 모듈을 읽지못하고 있어서 버셀 에러가 발생했습니다 수정부탁드립니다!
파일 : ./src/api/employer.ts:2:8
내용 : Type error: Module '"/vercel/path0/src/types/myShop"' has no default export. Did you mean to use 'import { RegisterFormData } from "/vercel/path0/src/types/myShop"' instead?
위치 : import RegisterFormData from '@/types/myShop';
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.
src/context/authProvider.tsx
Outdated
| const USER_ID_KEY = 'thejulge_user_id'; | ||
| const EXPIRES_KEY = 'thejulge_expires_at'; | ||
| const EXPIRES_DURATION_MS = 10 * 60 * 1000; // 10분 | ||
| const EXPIRES_DURATION_MS = 1000 * 60 * 1000; // 10분 |
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.
네 수정했습니다!
src/pages/my-shop/index.tsx
Outdated
| if (!user) | ||
| return ( | ||
| <Frame | ||
| title='내 가게' | ||
| content='로그인 후 가게를 관리할 수 있습니다.' | ||
| buttonText='로그인하기' | ||
| href='/login' | ||
| /> | ||
| ); |
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.
user가 알바생일 경우에는 해당 페이지가 보여지면 안돼니까 그냥 가장 상단에서 role 를 판별해서 사장님이아니면 메인페이지로 보내버리는게 어떨까요? 이부분 동작은 인화님과 동일하게 가져가야할것 같습니다!
(유저는 하나의 내가게로 인식할텐데 어디페이지에서는 노출하고, 어디페이지에서는 메인으로 보내버리므로 통일화 필요)
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.
네! 게스트일 경우 로그인화면으로 / 알바일 경우 메인페이지로 경고 모달과 함께 리다이렉트 시켰습니다.
src/pages/my-shop/index.tsx
Outdated
| </Notice> | ||
| <section className='mt-10 max-w-full bg-gray-100 tablet:mt-20'> | ||
| {shopNotice.length > 0 ? ( | ||
| <> |
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.
user 형태에 따라 모달 띄우는 것 때문에 return 내 구조를 조금 바꾸면서 해당 내용 바뀌었습니다!
src/pages/my-shop/index.tsx
Outdated
| {shopNotice.length > 0 ? ( | ||
| <> | ||
| <Container as='section' isPage className='pt-0'> | ||
| <div className='mt-7 tablet:mt-0'> |
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.
이부분 태그 없애고 container 에 넣을수도 있을것 같습니다!
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.
네 시멘틱태그 수정했습니다!

📝 작업 개요 (필수)
가게 등록 공고 조회 / 6개 이상 카드 무한 스크롤로 데이터 호출 구현
추가적으로, 제 페이지 작업하면서 포스트맨으로 테스트 마구 쐈던 공고들 어느정도 PUT 으로 수정해놨습니다😂
✨ 작업 내용 (필수)
📸 스크린샷
🧐 해결해야 하는 문제
🤔 리뷰어 확인 필요 사항
🔗 관련 이슈
🛠️ 후속 작업
✅ 체크리스트 (필수)