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: add storybook & designed component #262

Merged
merged 10 commits into from May 3, 2022

Conversation

Aylie-Chou
Copy link
Contributor

  • add component

    • button
      • pill button
    • text
      • headline
      • paragraph
    • icon
    • divider
  • notice

    • icons are stored in gcs, may occur same-origin problem
  • how to run storybook on local

    • yarn storybook

.circleci/config.yml Outdated Show resolved Hide resolved
.circleci/config.yml Outdated Show resolved Hide resolved
@Aylie-Chou Aylie-Chou force-pushed the 7-feature-add-storybook branch 2 times, most recently from a9edf8f to ff9bbe4 Compare April 28, 2022 09:10
.circleci/config.yml Outdated Show resolved Hide resolved
Aylie Chou added 10 commits May 3, 2022 13:47
- add component
  - button
    - pill button
  - text
    - headline
    - paragraph
  - icon
  - divider

- notice
  - icons are stored in gcs, may occur same-origin problem

- how to run storybook on local
  - `yarn storybook`
- circleci: storybook job only run on master branch
- storybook config: support deeper `stories` directory
- pillButton: support withIcon props
@Aylie-Chou
Copy link
Contributor Author

As previous discussion, remove ci config for this patch.
cc @taylrj

@taylrj
Copy link
Contributor

taylrj commented May 3, 2022

Nice work ☄️.💥

As previous discussion, because it is closely related to the work that @duidae is doing, I'd like to invite @duidae to review this PR before it will be merged.

@Aylie-Chou Aylie-Chou merged commit 3cfe2de into twreporter:master May 3, 2022
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.

None yet

2 participants