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

[ Feature/DEV-117 ] 스토리북 환경 개선 #177

Merged
merged 10 commits into from
Oct 9, 2024
Merged

Conversation

Happhee
Copy link
Collaborator

@Happhee Happhee commented Oct 8, 2024

🔥 Related Issues

https://linear.app/fewletter/issue/DEV-117/2차-관련-스토리북-개선

💜 작업 내용

  • 메인 페이지 > 워크북 카드 스토리 작성

✅ PR Point

  • 폴더 구조 변경
    원래 컴포넌트파일명/{컴포넌트명}.stories . 로 사용하기로 했는데, 해당 구조가 헷갈리는 것같아. story라는 폴더를 하나 만들었습니다!
    이부분 피드백 부탁드립니다 !!

  • 워크북 단일 카드 스토리 / 워크북 리스트 스토리 (모킹)
    WorkbookCard.stories.tsx에서는 단일 스토리를 보여주었고, WorkbookCardList.stories.tsx에서는 모킹 기반으로 컴포넌트를 보여주었습니다!

  • mdx 파일 작성
    해당 부분은 디자이너 분께 보여드릴 문서로 이해해주시면 될 것 같습니다!

👀 스크린샷 / GIF / 링크

https://32522858.24th-web-team-1-fe-storybook.pages.dev/

📚 Reference

Copy link

cloudflare-workers-and-pages bot commented Oct 8, 2024

Deploying 24th-web-team-1-fe-storybook with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3c073c1
Status: ✅  Deploy successful!
Preview URL: https://281ab658.24th-web-team-1-fe-storybook.pages.dev
Branch Preview URL: https://feature-dev-117.24th-web-team-1-fe-storybook.pages.dev

View logs

Copy link
Collaborator

Choose a reason for hiding this comment

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

mdx 파일을 따로 놓은 이유가 있나용??

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@soomin9106 이건 혹시나 디자이너 분께 설명이 필요하거나, 아니면 우리끼리 컴포넌트 명세서 작성하는 파일로 생각하면 될것 같아요.

@soomin9106
Copy link
Collaborator

soomin9106 commented Oct 9, 2024

UI 에 케이스가 많을 때 저희도 유용하게 코드를 확인 할 수 있을 것 같아용!! 저도 조금 더 알아보겠습니다 😄

@Happhee Happhee merged commit b7d8b9f into develop Oct 9, 2024
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants