Skip to content

Conversation

@devdeun
Copy link
Contributor

@devdeun devdeun commented Nov 28, 2024

🚀 풀 리퀘스트 제안

🔍 작업 내용

구독한 전략 페이지 작업했습니다!
기존에 수민님이 작성해주신 StrategiesModel이 api 수정 이슈로 변경되어서 함꼐 수정했습니다
로그인만 msw를 사용하도록 설정되어 있는데 스웨거 api가 아직 완성되지 않은 상태라 mockdata로 작업했습니다

🔧 변경 사항

  • 페이지네이션 로직 훅으로 분리

📸 스크린샷 (권장)

수정된 화면 또는 기능을 시연할 수 있는 스크린샷을 첨부해 주세요.

🙏 리뷰 참고 (선택 사항)

  • 일단 수민님이 말씀해 주신 부분은 다 수정한 것 같은데 혹시라도 빠진 부분이 있다면 알려주세요..!

📄 기타 (선택 사항)

그 외 전달하고 싶은 내용이나 특별한 요구 사항이 있으면 작성해 주세요.

@devdeun devdeun added 🔨 Refactor 코드 리팩토링 ✨ Feature 기능 개발 labels Nov 28, 2024
@devdeun devdeun self-assigned this Nov 28, 2024
@devdeun devdeun linked an issue Nov 28, 2024 that may be closed by this pull request
1 task
@github-actions
Copy link

🚀 Storybook이 배포 됐어요! 확인하러 가기 => https://6729e72ee61d8f57ca4790fb-lszycaekwe.chromatic.com/

@kimpra2989 kimpra2989 temporarily deployed to feat/favorites-page-151 - FinalProject_team3_FE PR #153 November 29, 2024 01:21 — with Render Destroyed
Copy link
Contributor

@ssumanlife ssumanlife left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다 !🥲
전략상세 쪽은 그대로인데 배포에러가 없네용... 다행... 배포에러는 없으니 일단 승인!!

basePath: PATH.STRATEGIES,
pageSize: COUNT_PER_PAGE,
})
const { data } = useGetStrategiesData({ isReady, page, size: COUNT_PER_PAGE })
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이것도 다시보니 size도 searchParams로 구하는게 좋겠네요,,,,

@kimpra2989 kimpra2989 temporarily deployed to feat/favorites-page-151 - FinalProject_team3_FE PR #153 November 29, 2024 01:22 — with Render Destroyed
@github-actions
Copy link

🚀 Storybook이 배포 됐어요! 확인하러 가기 => https://6729e72ee61d8f57ca4790fb-gmdszeqvmx.chromatic.com/

@github-actions
Copy link

🚀 Storybook이 배포 됐어요! 확인하러 가기 => https://6729e72ee61d8f57ca4790fb-fdlnquncyz.chromatic.com/

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

굿굿

@ghost ghost merged commit 2690568 into develop Nov 29, 2024
1 check passed
@ghost ghost deleted the feat/favorites-page-151 branch November 29, 2024 02:30
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 기능 개발 🔨 Refactor 코드 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] 구독한 전략 페이지

4 participants