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

Set up Storybook; create an Icon component as a proof of concept #6

Merged
merged 5 commits into from
Dec 19, 2024

Conversation

jessepinho
Copy link
Collaborator

@jessepinho jessepinho commented Dec 18, 2024

In this PR

  • Set up Storybook using react-native-web to render RN components on web
  • Create an <Icon /> component as a POC
  • Add the icon to the home screen to prove that it works in both the real app environment and on web

@jessepinho jessepinho force-pushed the jessepinho/initial-components branch from c32f545 to e5dfc45 Compare December 18, 2024 23:29
@jessepinho jessepinho force-pushed the jessepinho/initial-components branch from e5dfc45 to 17b88a2 Compare December 18, 2024 23:30
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

image

@jessepinho jessepinho marked this pull request as ready for review December 19, 2024 01:25
@jessepinho jessepinho requested a review from TalDerei December 19, 2024 01:25
Copy link
Collaborator

@TalDerei TalDerei left a comment

Choose a reason for hiding this comment

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

great work wrangling storybook support!

As we discussed offline, we should separately set up the necessary CI pipelines to generate preview Storybook URLs for pull requests.

@jessepinho jessepinho merged commit a4599f1 into main Dec 19, 2024
@jessepinho jessepinho deleted the jessepinho/initial-components branch December 19, 2024 18:02
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.

2 participants