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

chore: add storybook, basic story and documentation example #1

Merged
merged 7 commits into from
Jul 5, 2022

Conversation

storyworks
Copy link
Collaborator

@storyworks storyworks commented Jul 4, 2022

Currently I've created only a basic IonWrapper which wraps pages in .
We may later discover a need for an IconPageWrapper which also includes for smaller components.

I've modified the default introduction but kept it just as an example for how to write documentation

Will add Chromatic separately

@github-actions
Copy link

github-actions bot commented Jul 4, 2022

Warnings
⚠️ Is this PR related to a Basecamp issue? If so link it via the PR description.
⚠️ ❗ Big PR (52521 changes)

(change count - 52521): Pull Request size seems relatively large. If Pull Request contains multiple changes, split each into separate PR will helps faster, easier review.

Generated by 🚫 dangerJS against ca6c497

@@ -0,0 +1,77 @@

import React from 'react';
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

"danger": "^11.0.7",
"eslint": "^8.19.0",
"eslint-config-prettier": "^8.5.0",
"eslint-config-standard": "^17.0.0",
"eslint-plugin-import": "^2.26.0",
"eslint-plugin-n": "^15.2.4",
Copy link
Collaborator Author

@storyworks storyworks Jul 4, 2022

Choose a reason for hiding this comment

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

Only this import was added manually, storybook threw an error since eslint-config-standard depended on it but it was never imported

@storyworks storyworks added effort: 2 priority:2day type: chore updating folder structure, initializing projects etc; no production code change and removed documentation labels Jul 4, 2022
@storyworks storyworks added this to the MVP milestone Jul 5, 2022
Copy link
Contributor

@tataihono tataihono left a comment

Choose a reason for hiding this comment

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

can you do ts main and preview?

.eslintrc.js Outdated Show resolved Hide resolved
@storyworks
Copy link
Collaborator Author

can you do ts main and preview?

Can try but it's .js in core

@@ -0,0 +1,38 @@
// Imports the Storybook's configuration API
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

"include": ["src"]
"include": ["src"],

"ts-node": {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

To fix typescript issue with storybook main

@storyworks storyworks requested a review from tataihono July 5, 2022 04:09
@storyworks storyworks merged commit 5a26219 into main Jul 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort: 2 priority:2day type: chore updating folder structure, initializing projects etc; no production code change
Projects
None yet
2 participants