meteor / todos

The example app "Todos", written following the Meteor Guide
Other
535 stars 367 forks source link

Apollo branch needs to be updated to reflect current Apollo project. #242

Open pentatronix opened 7 years ago

pentatronix commented 7 years ago

The current Apollo branch is a year old and only covers a very simple integration of Apollo, specifically just the graphQL portion. This branch should be updated to include the kinds of integrations found here:

http://dev.apollodata.com/core/meteor.html

Thanks!

abernix commented 7 years ago

Thanks for identifying this! Is this something you'd be willing to help with, @pentatronix?

@hwillson @lorensr Is this something we should consider for todos? Should we outline a general plan here?

hwillson commented 7 years ago

I think it's a great idea. There was some discussion about this a while back in https://github.com/meteor/todos/pull/167. People are already familiar with how todos works, so it would be great to demonstrate how Apollo can be used alongside Meteor, while leveraging the familiarity of todos. Relating to that PR, I had started a new version of todos a while back that demonstrated how to use Apollo Client + Apollo Server + MySQL with Meteor. Unfortunately I didn't get far before being pulled into other things, but this is definitely something I'd like to re-visit. That being said, I'm not sure when I'll get to it so if you're interested in helping with this @pentatronix (or if anyone else is interested), that would be great!

lorensr commented 7 years ago

Yeah I'd base it off the react branch and use react-apollo and meteor/apollo

pentatronix commented 6 years ago

I'm swamped on my end (And also not a react aficionado) so I probably won't be much use here.

Considering the positioning of Meteor and Apollo, it would be good at least one current example of full usage kicking around.

hwillson commented 6 years ago

Quick update here - I'm currently working on this.