search

minderlabs / demo

Minder Demo App
1 stars 0 forks source link

Apollo #13

Closed richburdon closed 7 years ago

richburdon commented 7 years ago

Hopefully you're not reading this kind of thing at the moment. Not least of the reasons being that I have to admit your curiosity about Apollo was well founded. It is so much simpler and straight-forward than Relay.

Although again I had to reverse engineer it to understand it properly (although the docs are much better):

https://github.com/alienlaboratories/react-demos/blob/051801ba75e9c79c5320f2e807edddd0b5566e75/docs/kbase/apollo.png

I have most of the technical functionality of the relay demo (and more) now working (queries with fragments, mutations, paging, network interface, etc.) The killer feature is of course Redux; I had talked myself out of needing it (Relays docs imply it's obsolete and it's quite complicated), but I think essential.

I found a few bugs too, but got quick responses from the team.

I'll write up a summary (currently https://github.com/alienlaboratories/react-demos/blob/051801ba75e9c79c5320f2e807edddd0b5566e75/sub/apollo/js/client/main.js).

On a meta level: a) I think my desire to get something working really exacerbates the cognitive load of really absorbing these frameworks (GraphQL, then Relay, then Native and now Redux and Apollo). b) I discounted Apollo having the potential resources/support of Relay. Yet Relay 2 doesn't inspire much confidence.

BTW, on the side, I completed the stories (team roster and delete); but the inscrutable nature of Relay's caching system (and abundant problems given mutations) made me want to explore Apollo thoroughly. I think I can probably port the entire current app from Relay in a 5-story (2 days) so am inclined to do that -- esp. since in some respects (caching, queries) I'm further ahead in Apollo at this point.

RB

adamberenzweig commented 7 years ago

Finally read this, thanks for the notes here.

On Fri, Nov 11, 2016 at 7:51 PM, Rich Burdon notifications@github.com wrote:

Hopefully you're not reading this kind of thing at the moment. Not least of the reasons being that I have to admit your curiosity about Apollo was well founded. It is so much simpler and straight-forward than Relay.

Although again I had to reverse engineer it to understand it properly (although the docs are much better):

https://github.com/alienlaboratories/react-demos/blob/ 051801ba75e9c79c5320f2e807edddd0b5566e75/docs/kbase/apollo.png

I have most of the technical functionality of the relay demo (and more) now working (queries with fragments, mutations, paging, network interface, etc.) The killer feature is of course Redux; I had talked myself out of needing it (Relays docs imply it's obsolete and it's quite complicated), but I think essential.

I found a few bugs too, but got quick responses from the team.

I'll write up a summary (currently https://github.com/ alienlaboratories/react-demos/blob/051801ba75e9c79c5320f2e807eddd d0b5566e75/sub/apollo/js/client/main.js).

On a meta level: a) I think my desire to get something working really exacerbates the cognitive load of really absorbing these frameworks (GraphQL, then Relay, then Native and now Redux and Apollo). b) I discounted Apollo having the potential resources/support of Relay. Yet Relay 2 doesn't inspire much confidence.

BTW, on the side, I completed the stories (team roster and delete); but the inscrutable nature of Relay's caching system (and abundant problems given mutations) made me want to explore Apollo thoroughly. I think I can probably port the entire current app from Relay in a 5-story (2 days) so am inclined to do that -- esp. since in some respects (caching, queries) I'm further ahead in Apollo at this point.

RB

You can view, comment on, or merge this pull request online at:

https://github.com/alienlaboratories/react-demos/pull/13 Commit Summary

  • Apollo app with list query.
  • Clean-up
  • Apollo Search List.
  • Mutations.
  • Fragments.
  • Devtools
  • Paging/cursors.
  • Issue filed for paging bug.
  • Workaournd for paging bug.

File Changes

Patch Links:

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/alienlaboratories/react-demos/pull/13, or mute the thread https://github.com/notifications/unsubscribe-auth/AAWAUaoZ7lmcqwlsuxb61ZrKDw03zFazks5q9Q2kgaJpZM4KwRjG .

adamberenzweig commented 7 years ago

This is awesome. I'm psyched you're happy about Apollo and it looks like you've got some deep magic here already.

On Wed, Nov 16, 2016 at 7:23 PM, Rich Burdon notifications@github.com wrote:

Merged #13 https://github.com/alienlaboratories/react-demos/pull/13.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/alienlaboratories/react-demos/pull/13#event-862033398, or mute the thread https://github.com/notifications/unsubscribe-auth/AAWAUecaSe7xKIF0tbIT7HcihKQoIlJjks5q-56XgaJpZM4KwRjG .