-
Notifications
You must be signed in to change notification settings - Fork 5
Restructure project and add Storybook #84
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
Changes from all commits
6f7f55c
b6c0828
18ea196
f998ba1
04121fd
c4eaac6
3782bb5
25fc194
b757fcf
c78d974
77ce5c7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,3 @@ | ||
.idea | ||
/node_modules/ | ||
node_modules/ | ||
**/yarn-error.log |
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
@@ -0,0 +1,39 @@ | ||||||
# Thunderstore UI | ||||||
|
||||||
## Installation | ||||||
|
||||||
Run `yarn install` at the root of the project, that's it. | ||||||
|
||||||
## Repository structure | ||||||
|
||||||
This repository is intended to function as a monorepo, which contains several | ||||||
sub-projects inside it. The goal of this structure is to enable easier code | ||||||
sharing between projects as well as more efficient and higher coverage test | ||||||
suites | ||||||
|
||||||
Tools used for enabling this behavior are primarily: | ||||||
|
||||||
- yarn workspaces, as defined in the root `package.json` | ||||||
- typescript configuration inheritance and path aliases | ||||||
- tweaks for module resolution done in other tools such as nextjs or webpack, | ||||||
making sure the path aliased packages are appropriately resolved | ||||||
Comment on lines
+16
to
+19
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These should probably start with capital letters |
||||||
|
||||||
All sub-projects exist in the `packages` directory, includinig the following: | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
### components | ||||||
|
||||||
This project is intended to house common UI components that can be re-used | ||||||
across different projects, e.g. the Thunderstore Mod Manager as well as the | ||||||
website | ||||||
|
||||||
### nextjs | ||||||
|
||||||
This nextjs-based website project | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This sounds incomplete? |
||||||
|
||||||
### storybook | ||||||
|
||||||
[storybook](https://storybook.js.org/) is used to make the component catalogue | ||||||
easier to browse. | ||||||
|
||||||
You can run it with `yarn storybook` within the storybook project, and access it | ||||||
via localhost at the port provided on the terminal. |
This file was deleted.
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Each project currently ignores their own
node_modules
(other than the newstorybook
one). These should either be removed from the projects that include it, so it's clear that it is ignored at the root, or added tostorybook
and this change should be reverted.