-
Notifications
You must be signed in to change notification settings - Fork 0
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
storybook 및 chromatic 설정 #23
Conversation
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.
- 수고하셨습니다. 크로마틱이 있으면 필요하다고 생각했던 시점이었는데 덕분에 적절한 시점에 도입할 수 있게 되었네요.
- 한 가지 코멘트를 남겼습니다. 확인해보시고 문제가 없다면 바로 병합하셔도 될 것 같습니다.
package.json
Outdated
"lint-staged": "lint-staged", | ||
"storybook": "storybook dev -p 6006", | ||
"build-storybook": "storybook build", | ||
"chromatic": "npx chromatic --project-token=chpt_74ff4fb76a7b30e" |
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.
[comment]
이 project-token은 암호화할 필요가 없는 토큰인가요?
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.
storybook에서 script를 자동으로 설정해주어서 project-token이 노출되고 있는지 몰랐습니다ㅠ
금전적인 문제가 있진 않겠지만 project-token을 가지고 저희 프로젝트 chromatic에 아무나 story를 올릴 수 있는 문제가 있어서 project-token을 재발급 받았고 해당 script는 삭제했습니다!
참고로 pr 시 자동으로 빌드가 되기에 불필요하다고 생각해 삭제했습니다
저 궁금한 점이 생겼는데, RSC에 대한 스토리북 지원 여부나 이슈에 대해 혹시 설정하시면서 알게 된게 있으신가요? |
설정하면서 알게 된 건 아닌데 최근에 아티클을 통해 비동기나 모듈 모킹에 대해 storybook이 실험적인 기능을 제공한다는 걸 알게 되었어요! |
📌 작업 내용
storybook sample code를 작성했고 github actions 를 통해 pr을 올릴때마다 comment로 storybook 배포 링크를 첨부하도록 설정했습니다