Add apollo example with next.js

[donedgardo]

No description provided.

[donedgardo]

#387

[rauchg]

Not a big fan of submodules. Would it make sense to check it in statically?

[rauchg]

Thanks for the contribution!

[donedgardo]

Will do

[donedgardo]

@rauchg Should be done.
Thank you for the great work!
We'll be writing a medium article about your amazing work soon.

Btw travis ci build failed because no node env was set?

[rauchg]

Oh, seems like linting issues. Please run the linter on the new directory.

[rauchg]

npm run lint
https://github.com/zeit/next.js/blob/master/package.json#L31

[arunoda]

Run " npm run lint " in the main next.js directory rather in the example directory.

[adamsoffer]

Thanks @donedgardo - let's hold off on this? @sedubois and I have been hardening a simplified react apollo example over at https://github.com/ads1018/next-apollo-example

[donedgardo]

Fixed linting issues.

[rauchg]

I'd like to have a more minimalistic example, something that only does the apollo data wrapping (minus all the stylesheets, external fonts, etc). Maybe @ads1018 can contribute some ideas on how to make the most to-the-point Apollo example here?

[donedgardo]

I could simplify it with only rendering one component connected to apollo.

[adamsoffer]

@rauchg definitely. A lot of our thoughts have been captured towards the end of the apollo example issue. I'll follow up with a summary of the minimalistic approach we took in our example.

[adamsoffer]

@donedgardo @rauchg I ended up creating a separate pull request since I took a different approach. In my example, I simply wrap the page inside a higher-order component I created called withData. Using the HOC pattern I'm able to pass down a central store of query result data created by Apollo into the React component hierarchy defined inside the page it wraps.

On initial page load, while on the server and inside the HOC getInitialProps, I invoke the Apollo method, getDataFromTree. This method returns a promise; at the point in which the promise resolves, the Apollo Client store is completely initialized.

[timneutkens]

@donedgardo @ads1018 Thank you both so much for you contribution. This is pretty awesome. I think we should close one and work together on the best solution. From the looks of it #780 is more minimalistic. What do you guys think?

[sedubois]

The other example has been merged so I think this can be closed 👍

[timneutkens]

Done 👍🏻