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

Add Getting Started for Next.js & Server-Side Rendering docs #2381

Merged
merged 17 commits into from
Sep 14, 2020

Conversation

ericclemmons
Copy link
Contributor

@ericclemmons ericclemmons commented Sep 10, 2020

Issue #, if available:
aws-amplify/amplify-js#5435

Description of changes:

Screen Shot 2020-09-11 at 11 43 42
Screen Shot 2020-09-11 at 11 43 50
Screen Shot 2020-09-11 at 11 44 00
Screen Shot 2020-09-11 at 11 44 15
Screen Shot 2020-09-11 at 11 44 27
Screen Shot 2020-09-11 at 11 44 33
Screen Shot 2020-09-11 at 11 44 43

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ericclemmons ericclemmons marked this pull request as ready for review September 10, 2020 23:38
Copy link
Contributor

@harrysolovay harrysolovay left a comment

Choose a reason for hiding this comment

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

LGTM

@ericclemmons ericclemmons changed the title Add Getting Started for Next.js Add Getting Started for Next.js & Server-Side Rendering docs Sep 11, 2020
yarn test wasn't working locally and hung after pre-rendering
@mtliendo
Copy link

Been following this and the discussion post. Just wanted to say awesome work! This is super exciting!

@mtliendo
Copy link

mtliendo commented Sep 12, 2020

On the "Getting started with SSR" page, after the first paragraph, would it be beneficial to add a paragraph/diagram/link explaining why building for SSR is different than client-side only--or specifically why Amplify behaves differently.

That way those migrating from CRA -> NextJS for the first time have some context on why things are done differently.

docs/lib/ssr/fragments/js/getting-started.md Outdated Show resolved Hide resolved
docs/lib/ssr/fragments/js/getting-started.md Outdated Show resolved Hide resolved
docs/lib/ssr/ssr.md Outdated Show resolved Hide resolved
docs/start/getting-started/nextsteps.md Show resolved Hide resolved
Copy link
Contributor

@sammartinez sammartinez left a comment

Choose a reason for hiding this comment

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

LGTM 🌮

@ericclemmons ericclemmons merged commit 9d5c10f into master Sep 14, 2020
@ericclemmons ericclemmons deleted the next.js-ssr branch September 14, 2020 23:17
ericclemmons added a commit that referenced this pull request Sep 30, 2020
Regression from #2381 where the React fragment was intended to be shared for both React & Next.js
sammartinez pushed a commit that referenced this pull request Sep 30, 2020
Regression from #2381 where the React fragment was intended to be shared for both React & Next.js
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.

5 participants