@apollo/client compatibility

Hi! I tried apollo-link-debounce on @apollo/client, but the packages have slightly changed. In particular, ApolloLink is now part of @apollo/client (rather than apollo-link), and a few fields have been added (onError and setOnError, which seems to be very small functions - probably inheritable?).

[omerman]

@1v4nx published ur work to npm because I would like to use it right away… -> https://www.npmjs.com/package/omerman-apollo-link-debounce

[bennypowers]

While we wait for the PR, you can apply this patch with patch-package (this patch also happens to convert the file to standard modules for better browser support)

patches/apollo-link-debounce+2.1.0.patch

diff --git a/node_modules/apollo-link-debounce/build/dist/DebounceLink.d.ts b/node_modules/apollo-link-debounce/build/dist/DebounceLink.d.ts
index 06bf050..c48a961 100644
--- a/node_modules/apollo-link-debounce/build/dist/DebounceLink.d.ts
+++ b/node_modules/apollo-link-debounce/build/dist/DebounceLink.d.ts
@@ -1,5 +1,4 @@
-import { ApolloLink, Operation, NextLink } from 'apollo-link';
-import { Observable } from 'zen-observable-ts';
+import { ApolloLink, Operation, NextLink, Observable } from '@apollo/client/core ';
 export default class DebounceLink extends ApolloLink {
     private debounceInfo;
     private defaultDelay;
diff --git a/node_modules/apollo-link-debounce/build/dist/DebounceLink.js b/node_modules/apollo-link-debounce/build/dist/DebounceLink.js
index f52668b..f66932c 100644
--- a/node_modules/apollo-link-debounce/build/dist/DebounceLink.js
+++ b/node_modules/apollo-link-debounce/build/dist/DebounceLink.js
@@ -1,4 +1,3 @@
-"use strict";
 var __extends = (this && this.__extends) || (function () {
     var extendStatics = Object.setPrototypeOf ||
         ({ __proto__: [] } instanceof Array && function (d, b) { d.__proto__ = b; }) ||
@@ -9,10 +8,9 @@ var __extends = (this && this.__extends) || (function () {
         d.prototype = b === null ? Object.create(b) : (__.prototype = b.prototype, new __());
     };
 })();
-exports.__esModule = true;
-var apollo_link_1 = require("apollo-link");
-var zen_observable_ts_1 = require("zen-observable-ts");
-var DebounceLink = (function (_super) {
+import { ApolloLink } from "@apollo/client/link/core";
+import { Observable } from "@apollo/client/utilities";
+const DebounceLink = (function (_super) {
     __extends(DebounceLink, _super);
     function DebounceLink(defaultDelay) {
         var _this = _super.call(this) || this;
@@ -63,7 +61,7 @@ var DebounceLink = (function (_super) {
         if (!debounceKey) {
             return forward(operation);
         }
-        return new zen_observable_ts_1.Observable(function (observer) {
+        return new Observable(function (observer) {
             var debounceGroupId = _this.enqueueRequest({ debounceKey: debounceKey, debounceTimeout: debounceTimeout }, { operation: operation, forward: forward, observer: observer });
             return function () {
                 _this.unsubscribe(debounceKey, debounceGroupId, observer);
@@ -123,6 +121,6 @@ var DebounceLink = (function (_super) {
         dbi.currentGroupId++;
     };
     return DebounceLink;
-}(apollo_link_1.ApolloLink));
-exports["default"] = DebounceLink;
+}(ApolloLink));
+export default DebounceLink;
 //# sourceMappingURL=DebounceLink.js.map
\ No newline at end of file
diff --git a/node_modules/apollo-link-debounce/build/dist/index.d.ts b/node_modules/apollo-link-debounce/build/dist/index.d.ts
index bc94fdf..d45236d 100644
--- a/node_modules/apollo-link-debounce/build/dist/index.d.ts
+++ b/node_modules/apollo-link-debounce/build/dist/index.d.ts
@@ -1,2 +1,3 @@
 import DebounceLink from './DebounceLink';
+export { DebounceLink };
 export default DebounceLink;
diff --git a/node_modules/apollo-link-debounce/build/dist/index.js b/node_modules/apollo-link-debounce/build/dist/index.js
index 2f651f0..f5a365f 100644
--- a/node_modules/apollo-link-debounce/build/dist/index.js
+++ b/node_modules/apollo-link-debounce/build/dist/index.js
@@ -1,5 +1,3 @@
-"use strict";
-exports.__esModule = true;
-var DebounceLink_1 = require("./DebounceLink");
-exports["default"] = DebounceLink_1["default"];
-//# sourceMappingURL=index.js.map
\ No newline at end of file
+import DebounceLink from './DebounceLink.js';
+export { DebounceLink };
+export default DebounceLink;

By the way, I noticed you’re exporting CJS Modules with classes compiled down to ES5. Would you consider a further PR (after this one) that would change the output target to, say, ES2017 and the module format to standard ES Modules? That would help with bundle size and runtime performance. Legacy users would still be able to transpile at build-time as an app-level operation.