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

chore-fe: 스토리북 현재 의존성에 맞게 변경 #495

Merged
merged 5 commits into from
Aug 22, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Aug 17, 2024

Original issue description

목적

스토리북 현재 의존성에 맞게 변경

작업 세부사항

  • 스토리북 현재 의존성에 맞게 변경

참고 사항

  • 현재 여러 스토리북 파일이 터져있어요. React-Router 적용과, 컴포넌트 변경에 따른 수정이 빠진 경우가 생기면서 터졌는데, 이를 고치기 위함입니다.

아래의 별표줄 밑에 요구사항 ID만 작성해주세요. Prefix 금지!


CHORE_STORY

closes #494

스토리북의 useArgs는 공식적으론 Addon decorator와 호환이 되지 않습니다.
이번에 설치한 withRouter랑은 호환이 안되네요.
이는 스토리북이 리액트와 별도의 렌더링 싸이클을 거치기 때문입니다.


The reason for that is SB clears the storyContext as soon as the story is rendered. So during the first render it works, but on the next render, it breaks.

This has already been reported in [#33](JesusTheHun/storybook-addon-remix-react-router#33), and sadly things have not changed on the SB side.

You can see [storybookjs/storybook#12006](https://github.com/storybookjs/storybook/issues/12006) .


저는 추가적으로 이렇게 useArgs가 만들어진 것에 대한 납득이 갔습니다.
withRouter가 필요한 부분이라면 useArgs를 쓰지 않게 컴포넌트가 구현되어야 잘 만들어진 컴포넌트라고 봤습니다.
그래서 withRouter를 전역 decorator가 아니라 지역적으로 사용되도록 만들었습니다.

무슨말인지 모르시겠다면 정리해서 글 공유해드릴게요 ㅠㅠ
CDD 어렵다~

@github-actions github-actions bot added chore 빌드 업무 수정, 패키지 관리자 구성 등 업데이트, Production Code 변경 없음 frontend 프론트엔드 labels Aug 17, 2024
Copy link
Contributor Author

1723944778.121719

@lurgi lurgi marked this pull request as ready for review August 18, 2024 01:32
Copy link
Contributor Author

1723944781.326359

Copy link
Contributor Author

1723944789.458859

Copy link
Contributor

@llqqssttyy llqqssttyy left a comment

Choose a reason for hiding this comment

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

앱에 구현되어 있는 라우터를 생각하면 element로 들어가는 page에 prop을 전달하진 않으니 withRouter를 사용하는 곳이라면 useArgs를 사용할 일이 없어야 하는 게 맞는 것 같습니다.

원인을 파악하기 힘들었을 것 같은데 고생 많으셨습니다.👍

Copy link
Contributor

@seongjinme seongjinme left a comment

Choose a reason for hiding this comment

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

위에 렛서의 코멘트를 보고 이유를 어렴풋이 짐작하게 되었습니다만, 아직 잘 모르는 영역이라 추후에 설명을 부탁드려도 될까요? 고생하셨습니다 👍

@lurgi lurgi merged commit f26a3ee into fe/develop Aug 22, 2024
1 check passed
@lurgi lurgi deleted the fe-494-CHORE_STORY branch August 22, 2024 09:16
seongjinme pushed a commit that referenced this pull request Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore 빌드 업무 수정, 패키지 관리자 구성 등 업데이트, Production Code 변경 없음 frontend 프론트엔드
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

3 participants