vesper icon indicating copy to clipboard operation
vesper copied to clipboard

Fix #38 #40 resolver in subscription

Open josephktcheung opened this issue 6 years ago • 2 comments

This is to fix #38. Since there are no CurrentRequest, CurrentResponse and dataLoaders in the context / container when it's subscription, the resolver will fail when it tries to access those from context in buildCustomResolvers.

Since CurrentRequest and CurrentResponse can be undefined in subscriptions, I use the try catch to check if those 2 keys exists in the Container.

I also set dataLoaders to an empty array in context when a new subscriber connects to server.

josephktcheung avatar Sep 10 '18 14:09 josephktcheung

Also fix #40 by adding connectionParams to context so that user can use setUpContainer callback to implement authentication (similar to sample 10)

josephktcheung avatar Sep 11 '18 04:09 josephktcheung

Any update on this? This is an awesome framework! would be a shame to stop updating it

otonielguajardo avatar Jan 09 '19 20:01 otonielguajardo