Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix(subscription): update request return type correctly match apollo-link #727

Closed
wants to merge 2 commits into from
Closed

fix(subscription): update request return type correctly match apollo-link #727

wants to merge 2 commits into from

Commits on Jul 7, 2022

  1. fix(subscription) Updated request return type correctly match apollo-… 

    …link

Current type which doesn't compile.

```
    request(operation: Operation): Observable<unknown>;
```

New type after change which compiles

```
    request(operation: Operation): Observable<FetchResult> | null;
```
    @dpilch
    Mark Wolfe authored and dpilch committed Jul 7, 2022
    Configuration menu
    Copy the full SHA
    decf92f View commit details
    Browse the repository at this point in the history

  2. fix: cast observable

    @dpilch
    dpilch committed Jul 7, 2022
    Configuration menu
    Copy the full SHA
    8642f5b View commit details
    Browse the repository at this point in the history