-
Notifications
You must be signed in to change notification settings - Fork 26.4k
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
Implemented example for rbx and bulma-pro #7728
Conversation
Added example with rbx and bulma-pro package
Stats from current PRClick to expand stats
Click to expand serverless stats
|
Stats from current PRClick to expand stats
Click to expand serverless stats
|
Hi @Restoration , thank you for the PR, I took a look and I like it, there are still some changes required:
|
@@ -0,0 +1,16 @@ | |||
{ | |||
"name": "create-next-example-app", |
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.
"name": "create-next-example-app", | |
"name": "with-rbx-bulma-pro", |
<li>Automatic code splitting for faster page loads</li> | ||
<li>Simple client-side routing (page based)</li> | ||
<li>Webpack-based dev environment which supports Hot Module Replacement (HMR)</li> | ||
<li>Able to implement with Express or any other Node.js HTTP server</li> |
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.
<li>Able to implement with Express or any other Node.js HTTP server</li> | |
<li>Serverless support</li> |
<Layout> | ||
<Section> | ||
<Title as='h2'>About</Title> | ||
<p>That's exactly what we do with Next.js. Instead of PHP, we build the app with JavaScript and React. Here are some other cool features Next.js brings to the table:</p> |
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.
Why does it start like that?, makes no sense to me.
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.
I copied text from getting start guide
I'll fix it
<ul> | ||
<li>Server-rendered by default</li> | ||
<li>Automatic code splitting for faster page loads</li> | ||
<li>Simple client-side routing (page based)</li> |
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.
<li>Simple client-side routing (page based)</li> | |
<li>Filesystem based router, including dynamic routes</li> |
const ContactPage = () => ( | ||
<Layout> | ||
<Section> | ||
<Title as='h2'>Contact</Title> |
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.
<Title as='h2'>Contact</Title> | |
<Title as='h2'>Contact Form Example</Title> |
package.json
Outdated
@@ -107,5 +107,8 @@ | |||
}, | |||
"engines": { | |||
"node": ">= 8.0.0" | |||
}, | |||
"dependencies": { |
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.
Remove this please 🙏
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.
Update with the changes requested above
Stats from current PRClick to expand stats
Click to expand serverless stats
|
Stats from current PRClick to expand stats
Click to expand serverless stats
|
There is a typo in README. |
Requested [here](#7728 (comment))
Requested [here](#7728 (comment))
Yeah, sorry =) |
Added example with rbx and bulma-pro package