Fix: 알림, 검색, 상세페이지 디자인 및 로직 수정 #294
Conversation
# Conflicts: # src/app/search/page.tsx
# Conflicts: # src/styles/theme.css.ts
* Fix: step2 순위뱃지, 타이틀인풋 UI 깨짐 수정 * Design: ItemAccordion hr선 색상 변경 * Feat: 요청 주제 클릭 시 자동 제목 입력 추가 * Fix: 리스트 수정 게시 불가 오류 해결 - 의존성 제거 * Chore: 불필요 코멘트 삭제 * Chore: 불필요 주석 삭제
kanglocal
requested review from
Eugene-A-01,
Nahyun-Kang,
ParkSohyunee and
seoyoung-min
|
@kanglocal is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel. A member of the Team first needs to authorize it. |
ParkSohyunee
left a comment
ParkSohyunee
left a comment
There was a problem hiding this comment.
현지님~ 많은 작업 해주셔서 감사합니다. 너무 고생많으셨습니다. 👍✨🥰 필수로 수정해야하는 부분에는 📌로 표시해두었는데 이부분 확인 부탁드립니다!! LGTM!!! 🎉🎉
There was a problem hiding this comment.
📌 현지님! 리액션 api 파일이름 소문자로 변경 및 updateReaction.ts가 더 적절할 것 같아요!
There was a problem hiding this comment.
유틸함수에 dateFormat함수와 거의 비슷한 기능을 하는 것 같아서 해당 파일로 옮기거나 하나의 함수에서 사용하는 것으로 리팩토링하면 좋을 것 같습니다!
| // export interface NotificationsType { | ||
| // alarmList: NotificationType[]; | ||
| // } |
There was a problem hiding this comment.
📌 CI/CD에러가 여기서 나는 것 같아서 해당 부분 주석을 제거하거나, 해당 타입을 사용하는 곳에서 NotificationType[]으로 타입을 변경해야할 것 같습니다~!
There was a problem hiding this comment.
@kanglocal 현지님 이거 주석 해제만 하여 커밋해 주실 수 있나요?! 그러면 머지가 가능할 것 같습니다!
|
|
||
| async function getSearchUserResult({ keyword }: GetSearchUserResultType) { | ||
| async function getSearchUserResult({ keyword, page }: GetSearchUserResultType) { | ||
| console.log('page:::', page); |
There was a problem hiding this comment.
📌 콘솔로그 삭제해 주시면 좋을 것 같습니당
| import { useLanguage } from '@/store/useLanguage'; | ||
| import formatDateTime from '@/lib/utils/dateTimeFormat'; | ||
| import LockIcon from '*.svg'; | ||
| import { vars } from '@/styles/__theme.css'; |
There was a problem hiding this comment.
📌 버전2.0 테마가 사용되고 있는 것 같아요!!
| }; | ||
| }, [data.reactions]); | ||
|
|
||
| // TODO: 현재 새로굄해야만 반영되는 버그 있음. 낙관적업데이트도 되지않고있음. 수정필요. |
| const ReactionIcon = ({ type, ...props }: { type: ReactionType } & React.SVGProps<SVGSVGElement>) => { | ||
| switch (type) { | ||
| case 'COOL': | ||
| return <ReactionGoodIcon {...props} />; | ||
| case 'AGREE': | ||
| return <ReactionAgreeIcon {...props} />; | ||
| case 'THANKS': | ||
| return <ReactionThanksIcon {...props} />; | ||
| default: | ||
| return null; | ||
| } | ||
| }; |
There was a problem hiding this comment.
리팩토링 시 ui 컴포넌트로 따로 분리해주면 좋을 것 같습니다~!!
There was a problem hiding this comment.
바텀시트 폴더 ver3.0에서 일반 바텀시트랑 컨펌바텀시트 모두 사용가능한 것으로 알고있는데 해당 부분 머지 후 바텀시트 작업할때 한번 확인해보겠습니다~!
3 participants
개요
작업 사항
참고 사항 (optional)
추가 작업 필요
버그
관련 이슈 (optional)
스크린샷
리뷰어에게