Skip to content

Instantly share code, notes, and snippets.

@n1ru4l
Last active September 21, 2020 14:34
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save n1ru4l/386fc9c7ad3e5528f4d0231e821103c9 to your computer and use it in GitHub Desktop.
Save n1ru4l/386fc9c7ad3e5528f4d0231e821103c9 to your computer and use it in GitHub Desktop.
Correct graphql-subscriptions asyncIterator typings
diff --git a/node_modules/graphql-redis-subscriptions/dist/redis-pubsub.d.ts b/node_modules/graphql-redis-subscriptions/dist/redis-pubsub.d.ts
index cc2e21e..870a2b8 100644
--- a/node_modules/graphql-redis-subscriptions/dist/redis-pubsub.d.ts
+++ b/node_modules/graphql-redis-subscriptions/dist/redis-pubsub.d.ts
@@ -13,7 +13,7 @@ export declare class RedisPubSub implements PubSubEngine {
publish<T>(trigger: string, payload: T): Promise<void>;
subscribe(trigger: string, onMessage: Function, options?: Object): Promise<number>;
unsubscribe(subId: number): void;
- asyncIterator<T>(triggers: string | string[], options?: Object): AsyncIterator<T>;
+ asyncIterator<T>(triggers: string | string[], options?: Object): AsyncIterableIterator<T>;
getSubscriber(): RedisClient;
getPublisher(): RedisClient;
close(): void;
diff --git a/node_modules/graphql-subscriptions/dist/pubsub-engine.d.ts b/node_modules/graphql-subscriptions/dist/pubsub-engine.d.ts
index 58fa1b1..c8168e5 100644
--- a/node_modules/graphql-subscriptions/dist/pubsub-engine.d.ts
+++ b/node_modules/graphql-subscriptions/dist/pubsub-engine.d.ts
@@ -2,5 +2,5 @@ export declare abstract class PubSubEngine {
abstract publish(triggerName: string, payload: any): Promise<void>;
abstract subscribe(triggerName: string, onMessage: Function, options: Object): Promise<number>;
abstract unsubscribe(subId: number): any;
- asyncIterator<T>(triggers: string | string[]): AsyncIterator<T>;
+ asyncIterator<T>(triggers: string | string[]): AsyncIterableIterator<T>;
}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment