-
Notifications
You must be signed in to change notification settings - Fork 136
Observable Subscriptions #80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Observable Subscriptions #80
Conversation
|
Looks pretty good. I will take a closer look in the following days. |
|
And about the integration tests, probably we are going to move to use our own integrated server and not something exposed to internet |
|
The integration tests are already using the "IntegrationTestServer" in the tests subdirectory (since I did not find any public demo APIs which support subscriptions). So basically this server project can also be reused for all other tests which currently run against a public API. For everything NOT using websockets we can do this the way pointed out in this Microsoft tutorial With |
|
@rose-a Very well done. Many thanks. ( Paypal and I will give you the beer I promised ;) ) |
|
Can you create a new alpha with this included? I would like to use it. |
Hi, it seems like CreateSubscriptionStream is not a member function of GraphQL.Client.Http.GraphQLHttpClient I am using GraphQL (3.0.0-preview-1271) , GraphQL.Client(2.0.0-alpha.3) , GraphQL.Common (2.0.0-alpha.3). Could you please send me sample link. Appreciate your help. Thanks, |
Relating to #25
@deinok Here is my first stab at creating a subscription "API" based on
System.Reactive.I've left the existing API untouched and implemented my idea alongside it. I've also added a simple integration test based on the chat sample to prove it working.
Currently the new variant could be used as follows:
Disposing the parent
GraphQlHttpClientwill cancel and dispose all web sockets made on it, which will result in the subscribers receiving anOnError()call with aThreadCancelledException.This should satisfy the requirements you mentioned in #25.
Please let me know what you think.