-
-
Notifications
You must be signed in to change notification settings - Fork 134
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
Website #646
Website #646
Conversation
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.
@trace2798 Thanks for this PR 🙏
Here're some suggestion:
- Rename
website
folder todocs
- Edit the root
package.json
of main project and add the following inworkspaces
entry:
{
"workspaces": {
"packages": [
"packages/*",
"examples/*",
"docs"
]
}
website/package.json
Outdated
@@ -0,0 +1,30 @@ | |||
{ | |||
"name": "website", |
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.
Set name: "docs"
website/next.config.js
Outdated
@@ -0,0 +1,4 @@ | |||
/** @type {import('next').NextConfig} */ | |||
const nextConfig = {} |
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.
Make the website static by adding output
prop
/** @type {import('next').NextConfig} */
const nextConfig = {
output: 'export',
}
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.
On it
docs/package.json
Outdated
"typescript": "5.0.4" | ||
}, | ||
|
||
"workspaces": { |
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.
Not here, add it on root package.json
. This file
Lines 10 to 15 in cd9992b
"workspaces": { | |
"packages": [ | |
"packages/*", | |
"examples/*" | |
] | |
}, |
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.
My bad got it now tho
docs/yarn.lock
Outdated
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.
@trace2798 Remove this file in favour of root yarn.lock. Run install commands on root directory
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.
@trace2798 Thanks for this PR! Approved 🙏
Added the website. Working on docs and example layout