-
Notifications
You must be signed in to change notification settings - Fork 6
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
Conversation
1723944778.121719 |
1723944781.326359 |
1723944789.458859 |
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.
앱에 구현되어 있는 라우터를 생각하면 element로 들어가는 page에 prop을 전달하진 않으니 withRouter를 사용하는 곳이라면 useArgs를 사용할 일이 없어야 하는 게 맞는 것 같습니다.
원인을 파악하기 힘들었을 것 같은데 고생 많으셨습니다.👍
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.
위에 렛서의 코멘트를 보고 이유를 어렴풋이 짐작하게 되었습니다만, 아직 잘 모르는 영역이라 추후에 설명을 부탁드려도 될까요? 고생하셨습니다 👍
Co-authored-by: Jeongwoo Park <[email protected]>
Original issue description
목적
작업 세부사항
참고 사항
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 어렵다~