graphql-sequelize - Githubissues

benawad / graphql-express-template

Node.js GraphQL Server template

525 stars 195 forks source link

Closed ravishivt closed 7 years ago

ravishivt commented 7 years ago

I watched your video playlist for this repo and it's been very helpful. Thanks for working on it. A couple thoughts:

The YouTube playlist is out of order. The order of videos 1, 2, and 3 should be reversed.
What are your thoughts on graphql-sequelize? It seems like a good fit since you are using sequelize. I'm trying to evaluate whether to use it or not in my project.

I'm looking forward to your future videos, especially on how to leverage DataLoader and how to add more authorization/permission logic.

benawad commented 7 years ago

Hey, I'm glad the videos were helpful!

The order it shows for me is Installing..., Connect..., then Mutations... (which seems like the right order). How do you suggest it be ordered? Mutations..., Connect..., Installing...?
graphql-sequelize looks very interesting and seems like it would cut out a lot of boilerplate code. I haven't tried it yet, but I will soon and then I can give you a better opinion on it.
DataLoader/Authorization/Permission stuff is coming soon and it's going to be really really cool :)

ravishivt commented 7 years ago

My bad on #1. I linked the wrong playlist. This is the one I was referring to on being out of order.

benawad commented 7 years ago

You're totally right! Thanks for bringing this to my attention, I'll fix it.