-
Notifications
You must be signed in to change notification settings - Fork 4
my-profile 페이지네이션/알림 모달 연동 #88
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
|
@jeschun 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.
고생하셨습니다 코멘트만 확인 후 반영 부탁드립니다!
| 'text-[var(--gray-400)]': read, | ||
| })} | ||
| > | ||
| {`${shopName} (${DATE_RANGE.date} ${DATE_RANGE.startTime} ~ | ||
| ${DATE_RANGE.endTime}) 공고 지원이 `} | ||
| <span | ||
| className={clsx({ | ||
| 'text-gray-500': read, | ||
| 'text-blue-200': !read && result === 'accepted', | ||
| 'text-red-400': !read && result === 'rejected', | ||
| 'text-[var(--gray-500)]': read, | ||
| 'text-[var(--blue-200)]': !read && result === 'accepted', | ||
| 'text-[var(--red-500)]': !read && result === 'rejected', |
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.
혹시 컴포넌트의 컬러를 var-- 형태로 변환하신 이유가 있으실까요 ?
왜냐면 기존과 동일한 문법인데 바꾸신 이유가 있을까 해서요!
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-profile/index.tsx
Outdated
| <Table | ||
| headers={headers} | ||
| data={paged} | ||
| data={pagedRows} |
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.
인화님께서 해당 부분 props 를 수정하여서 인화님 파일 merge 후 수정하시고 재 Push 부탁드리겠습니다!
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.
table 컴포넌트를 인화님도 수정하신 상태여서 PR merge 후 재 push 부탁드립니다
📝 작업 개요 (필수)
알림 모달 제어모드 추가 – Notification에 isOpen/onClose 지원(헤더에서 직접 열·닫기).
헤더 종 버튼 단일화 – nav.tsx에서만 종 노출·제어, 내부 중복 트리거 제거.
알림 아이콘 상태 반영 – 미확인(승인/거절) 존재 시 notificationOn, 없으면 notificationOff.
신청 내역 페이지네이션 연결 – total/limit/offset/onPageChange 연동, 목록 변동 시 offset=0 리셋.
알림 메시지 가독성 – 색상 적용.
테이블 타이틀 통일 – “신청 내역” 타이포를 text-heading-s font-semibold로 정리.
✨ 작업 내용 (필수)
📸 스크린샷
🧐 해결해야 하는 문제
🤔 리뷰어 확인 필요 사항
notification.tsx: 제어모드 도입으로 헤더와 역할 분리(중복 종 제거, 책임 명확).
nav.tsx: 최소 변경으로 열림 상태만 관리, 읽지 않은 알림 개수로 아이콘 전환.
my-profile/index.tsx: 신청 내역 페이징 안정화(데이터 변경 시 첫 페이지로 리셋).
NotificationMessage.tsx: 색상 토큰 적용으로 상태 전달 명확.
table.tsx: 섹션 타이틀만 팀 공통 타이포로 정리(기능 변경 없음).
🔗 관련 이슈
🛠️ 후속 작업
✅ 체크리스트 (필수)