Skip to content
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

fix: 사용량 통계 부분 렌더링을 개선한다 #864

Merged
merged 5 commits into from
Oct 16, 2023
Merged

Conversation

feb-dain
Copy link
Collaborator

@feb-dain feb-dain commented Oct 16, 2023

📄 Summary

사용량 통계 부분 렌더링 개선한다.

  • 급속만 있는 충전소의 렌더링 문제 (-1 되었다가 반영)
  • 버튼 렌더링 최적화

🕰️ Actual Time of Completion

1시간

🙋🏻 More

close #863

🚀 Storybook

https://storybook.carffe.in/

@feb-dain feb-dain added 🚑 버그 예기치 않은 문제 또는 의도하지 않은 동작입니다 🛠️ 리팩터링 개선사항입니다 FE 프론트엔드 관련 이슈입니다 labels Oct 16, 2023
@feb-dain feb-dain self-assigned this Oct 16, 2023
@gabrielyoon7 gabrielyoon7 changed the title fix: 사용량 통계 부분 렌더링 개선한다 fix: 사용량 통계 부분 렌더링을 개선한다 Oct 16, 2023
@feb-dain feb-dain temporarily deployed to test October 16, 2023 07:06 — with GitHub Actions Inactive
@@ -53,7 +53,7 @@ const StationDetailsView = ({ station }: StationDetailsViewProps) => {

<ChargerList chargers={chargers} stationId={stationId} reportCount={reportCount} />

<CongestionStatistics stationId={stationId} />
<CongestionStatistics stationId={stationId} chargers={chargers} />
Copy link
Member

Choose a reason for hiding this comment

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

chargers를 리액트 쿼리에서 직접 가져오는 것은 어떤가요?! 나중에 CongestionStatistics 컴포넌트를 독립적으로 사용하게 되는 경우엔 chagers를 컴포넌트 내부에서 가져오는 것이 더 좋을 것 같다는 생각이 드네요!


import type { Charger } from '@type';

export const getChargerCountsAndAvailability = (chargers: Charger[]) => {
Copy link
Member

Choose a reason for hiding this comment

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

이거 좋네요 👍

@feb-dain feb-dain temporarily deployed to test October 16, 2023 07:34 — with GitHub Actions Inactive
Copy link
Member

@kyw0716 kyw0716 left a comment

Choose a reason for hiding this comment

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

무섭네요 ㅠㅠ

@feb-dain feb-dain merged commit b986dee into develop Oct 16, 2023
3 checks passed
@feb-dain feb-dain deleted the fix/863 branch October 16, 2023 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 관련 이슈입니다 🚑 버그 예기치 않은 문제 또는 의도하지 않은 동작입니다 🛠️ 리팩터링 개선사항입니다
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

사용량 통계 부분 렌더링 개선한다
3 participants