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

feat(web-storybook): create storybook setup & refactor for separation of concerns #5570

Merged
merged 8 commits into from
Jan 23, 2025

Conversation

sulmoJ
Copy link
Member

@sulmoJ sulmoJ commented Jan 20, 2025

Skip Review (optional)

  • Minor changes that don't affect the functionality (e.g. style, chore, ci, test, docs)
  • Previously reviewed in feature branch, further review is not mandatory
  • Self-merge allowed for solo developers or urgent changes

Description (optional)

I have created a Storybook for the console components.
To achieve this, I have separated the concerns within the repository into distinct packages:

  • postcss-config-custom

    • This package includes default configurations for TailwindCSS.
  • mirinae-foundation

    • This package manages the foundational style settings for all packages.
  • storybook-config-custom

    • This package contains the base configuration settings for Storybook.

콘솔 컴포넌트를 위한 스토리북을 생성했습니다.
이를 위해 현재 레포의 패키지 관심사 분리를 진행하였습니다.

  • postcss-config-custom
    • tailwindcss 기본 설정값을 추가하였습니다.
  • mirinae-foundation
    • 모든 패키지의 스타일 기본 값(디자인 토큰)들을 관리합니다.
  • storybook-config-custom
    • 스토리북 기본 세팅값을 분리하였습니다.

Things to Talk About (optional)

Copy link

vercel bot commented Jan 20, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
console ⬜️ Ignored (Inspect) Visit Preview Jan 23, 2025 7:37am
dashboard ⬜️ Ignored (Inspect) Visit Preview Jan 23, 2025 7:37am

@github-actions github-actions bot added pass/signedoff pending/review Pull Request is awaiting review. No reviewers have started reviewing yet. labels Jan 20, 2025
Copy link
Member

@WANZARGEN WANZARGEN left a comment

Choose a reason for hiding this comment

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

Wow. Thanks for your hard work!! Kudos!!
I left some suggestions, and let's discuss.

Copy link
Member

Choose a reason for hiding this comment

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

suggestion: What about to move it to mirinae-foundation package?

Copy link
Member

Choose a reason for hiding this comment

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

suggestion: I think it's far better to move it to the mirinae-foundation package

@sulmoJ sulmoJ force-pushed the develop branch 2 times, most recently from cbe0c12 to f82d953 Compare January 23, 2025 05:04
Copy link
Member

@WANZARGEN WANZARGEN left a comment

Choose a reason for hiding this comment

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

LGTM!!!!

dependabot bot and others added 7 commits January 23, 2025 16:24
Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 4.5.3 to 4.5.6.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/v4.5.6/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v4.5.6/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>
refactor: restructure styles and update imports

- Moved style files to a new package directory
- Updated import paths for colors, screens, and fonts
- Added mirinae-foundation as a dependency in package.json

Signed-off-by: sulmo <[email protected]>

refactor(tailwind): simplify config by extending default settings

- Replaced custom Tailwind configuration with an extended default config
- Removed redundant theme, variants, and plugins definitions
- Updated content paths for style processing

Signed-off-by: sulmo <[email protected]>
- Removed `@storybook/theming` dependency and replaced with custom theme creation.
- Updated import paths and methods for setting Storybook configurations.
- Adjusted package.json to reflect changes in dependencies.
- Simplified path resolution in storybook-config-custom.

Signed-off-by: sulmo <[email protected]>
- Change the development server port from 6006 to 6007 in the script configuration.

Signed-off-by: sulmo <[email protected]>
@sulmoJ sulmoJ merged commit af4af3f into cloudforet-io:develop Jan 23, 2025
5 checks passed
@sulmoJ sulmoJ deleted the develop branch January 23, 2025 07:52
@github-actions github-actions bot removed the pending/review Pull Request is awaiting review. No reviewers have started reviewing yet. label Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants