Skip to content

#1757 Relay Modern Example #2696

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

Merged
merged 1 commit into from
Aug 13, 2017
Merged

#1757 Relay Modern Example #2696

merged 1 commit into from
Aug 13, 2017

Conversation

petrvlcek
Copy link
Contributor

I've taken a working solution of SSR with Relay Modern from @steida (contributed here) and compiled it into more lightweight example.

Copy link
Member

@timneutkens timneutkens left a comment

Choose a reason for hiding this comment

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

Very nice!

@timneutkens timneutkens merged commit 62ed18d into vercel:v3-beta Aug 13, 2017
@steida
Copy link
Contributor

steida commented Aug 13, 2017

It's should probably have been merged to master, no?

@petrvlcek
Copy link
Contributor Author

Hi, @timneutkens, since 3.0 has been officialy released, this PR would probably belong to master branch. I'm not sure what is going to happen with v3-beta branch but I gues it is obsolete now.

@timneutkens
Copy link
Member

Definitely should be added to master, I'll merge it whenever I find the time 👍 Or you could create a new PR and i'll merge it 😄 ❤️

@petrvlcek
Copy link
Contributor Author

@timneutkens OK, a new PR is here: #2773 👍

timneutkens added a commit that referenced this pull request Aug 27, 2017
* Add examples/with-redux-code-splitting. (#2721)

* #1757 Relay Modern Example (#2696)

* Add ReasonML example (#2640)

* Add ReasonML example

* Add a gitignore specifically for the reasonml example

* Allow custom className for <Main /> (#2802)

* 3.0.2

* Remove beta information from the README.

* 3.0.3

* Remove unnecessary lookup in example with emotion (#2731)

* Document SCSS/Less (#2742)

* Document SCSS/Less

* Add missing word

* Add docs for examples dir

* Add extra example

* uppercase J

* Add with pkg example (#2751)

* Add custom server micro example (#2750)

* Ease running multiple examples at the same time with process.env.PORT (#2753)

* Add line-height rule for error page h2 (#2761)

* Add support for fetching multiple translation files (#2743)

* Add support for fetching multiple translation files

* Cleanup

* Clear missed interval (#2611)

* clear missed interval

* remove trailing whitespace

* Relay Modern Example (#1757) (#2773)

* Simplification of Relay Modern Example (#1757) (#2776)

* Use deterministic names for dynamic import (#2788)

* Always use the same name for the same dynamic import.

* Add unit tests for the modulePath generation.

* Allow tests to run correctly on Windows.

* Make the chunk name a bit pretty.

* Fix tests to run on Windows.

* 3.0.4

* Revert "Make the chunk name a bit pretty." (#2792)

This reverts commit 0c9e8cf.

* 3.0.5

* Use _ as the divider for dynamic import name splitter. (#2793)

Using - gives us some weird webpack errors.

* 3.0.6

* next/dynamic Error Message Tweaks (#2798)

* Fixed issue (#2804)

#2800

* docs(material-ui): move the source code to Material-UI repository (#2808)
anaisbetts pushed a commit to anaisbetts/next.js that referenced this pull request Sep 5, 2017
@lock lock bot locked as resolved and limited conversation to collaborators Aug 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants