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

storybook 및 chromatic 설정 #23

Merged
merged 7 commits into from
Feb 16, 2024
Merged

storybook 및 chromatic 설정 #23

merged 7 commits into from
Feb 16, 2024

Conversation

gahyuun
Copy link
Member

@gahyuun gahyuun commented Feb 15, 2024

📌 작업 내용

구현 내용 및 작업 했던 내역

  • storybook 및 chromatic 설정

storybook sample code를 작성했고 github actions 를 통해 pr을 올릴때마다 comment로 storybook 배포 링크를 첨부하도록 설정했습니다

Copy link

yougyung
yougyung previously approved these changes Feb 15, 2024
seonghunYang
seonghunYang previously approved these changes Feb 15, 2024
Copy link
Collaborator

@seonghunYang seonghunYang left a 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"
Copy link
Collaborator

Choose a reason for hiding this comment

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

[comment]
이 project-token은 암호화할 필요가 없는 토큰인가요?

Copy link
Member Author

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 시 자동으로 빌드가 되기에 불필요하다고 생각해 삭제했습니다

@gahyuun gahyuun dismissed stale reviews from seonghunYang and yougyung via 2eb4b61 February 15, 2024 11:01
Copy link

yougyung
yougyung previously approved these changes Feb 15, 2024
seonghunYang
seonghunYang previously approved these changes Feb 15, 2024
@seonghunYang
Copy link
Collaborator

저 궁금한 점이 생겼는데, RSC에 대한 스토리북 지원 여부나 이슈에 대해 혹시 설정하시면서 알게 된게 있으신가요?

@gahyuun
Copy link
Member Author

gahyuun commented Feb 15, 2024

저 궁금한 점이 생겼는데, RSC에 대한 스토리북 지원 여부나 이슈에 대해 혹시 설정하시면서 알게 된게 있으신가요?

설정하면서 알게 된 건 아닌데 최근에 아티클을 통해 비동기나 모듈 모킹에 대해 storybook이 실험적인 기능을 제공한다는 걸 알게 되었어요!
https://soobing.github.io/react/storybook-react-server-components/

@gahyuun gahyuun dismissed stale reviews from seonghunYang and yougyung via 94c8478 February 15, 2024 15:47
Copy link

@gahyuun gahyuun merged commit e6421ca into main Feb 16, 2024
3 checks passed
@gahyuun gahyuun deleted the setting-storybook/#21 branch February 16, 2024 01:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants