Skip to content

Conversation

@Joezo
Copy link
Contributor

@Joezo Joezo commented Mar 7, 2019

What does this PR do?

  • Adds an example application that @bmullan91 had been using for local dev. It's very simple but showcases the different functional features nicely and will provide a good base for local development.

  • Uses lerna's --hoist option so that multiple packages use the same copy of react/react-dom

  • Simplifies the eslint and prettier setup.

Related issues

closes #69

Checklist

  • I have checked the contributing document
  • I have added or updated any relevant documentation
  • I have added or updated any relevant tests

@Joezo Joezo requested review from bmullan91 and jackdclark March 7, 2019 11:22
@Joezo Joezo changed the title Add example SSR app built using Fastify chore: add example SSR app built using Fastify Mar 7, 2019
bmullan91
bmullan91 previously approved these changes Mar 7, 2019
Copy link
Contributor

@bmullan91 bmullan91 left a comment

Choose a reason for hiding this comment

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

Only a few small things 👍 Thanks for this @Joezo

Copy link
Contributor

@jackdclark jackdclark left a comment

Choose a reason for hiding this comment

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

Nice work @Joezo & @bmullan91

Copy link
Contributor

@jackdclark jackdclark left a comment

Choose a reason for hiding this comment

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

I'm sure we can make this look nicer, but it's a good foundation for local dev 👍

Download the example:

```bash
curl https://codeload.github.com/nearform/graphql-hooks/tar.gz/master | tar -xz --strip=2 graphql-hooks-master/examples/fastify-ssr
Copy link
Contributor

Choose a reason for hiding this comment

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

Worth mentioning that it can just be run from the examples/ dir if you already have graphql-hooks locally?

We should update CONTRIBUTING.md to these steps in a getting started section as @jh3y mentioned in #69 too. Not a blocker for this PR, can be followed up.

@Joezo
Copy link
Contributor Author

Joezo commented Mar 7, 2019

I'm sure we can make this look nicer, but it's a good foundation for local dev 👍

@jackdclark no it's beautiful as it is

jackdclark
jackdclark previously approved these changes Mar 7, 2019
@Joezo Joezo dismissed stale reviews from jackdclark and bmullan91 via d51b302 March 7, 2019 15:34
@Joezo Joezo merged commit 1cc8854 into master Mar 7, 2019
@Joezo Joezo deleted the example-app branch March 7, 2019 15:37
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.

Local set up included in contributing guidelines/getting started

3 participants