-
Notifications
You must be signed in to change notification settings - Fork 0
[feat] 분야별 내역 페이지 UI 구현 #26
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
base: main
Are you sure you want to change the base?
Conversation
dohy-eon
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.
수고하셨어요! 메인 페이지에서 총 지출액 클릭 시 /asset/sector-full로 이동하고, 그외 클릭 시 location.state를 통해 필터 상태를 넘겨주는 방식이 되게 좋아보입니당 깔끔해서 맘에 들어요~~
지금 단계에서 하면 좋을 것 같은거 한가지 정도만 말씀드릴게요! 현재 상세 페이지(SectorDetailPage)에서 카테고리 데이터를 찾기 위해 다시 전체 데이터를 가공(transformToCategoryGroups)하는 중인데, 만약 데이터가 크다면 이전 페이지에서 클릭한 sector 객체 전체를 Maps의 state로 넘겨주거나 store를 따로 파면 처리가 더 깔끔해질 것 같아요! 보시고 내일 회의 전까지 가능하시다면 작업해주시면 좋을 듯 합니다.
아래 벨로그 글이 짧고 간단해서 한번 슥 읽어보시고 모르겠는 부분들 검색하시면 좋을거에요!!
벨로그
|
주신 피드백 적용해서 코드 수정했습니다. 작업 목적
주요 변경 사항
|
month-test.mp4[작업 내용]
|
|
[작업 내용]
|
dohy-eon
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.
lgtm
dohy-eon
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.
34feb70 to
01b6d88
Compare
01b6d88 to
20ef73d
Compare
7f8b598
20ef73d to
7f8b598
Compare
|
커밋 메시지 수정 후 누락된 코드 추가 커밋했습니다. |
Pull Request
📝 변경 내용
🔗 관련 이슈
🎯 변경 사항
📱 테스트
📸 스크린샷 (UI 변경시)
test.mp4
📋 체크리스트
npm run format)npm run lint:fix)💻 코드 품질 확인
PR 제출 전에 다음 명령어를 실행하여 코드 품질을 확인해주세요: