Skip to content

Commit

Permalink
Merge pull request #1133 from DimitryDushkin/fix-swallow-errors
Browse files Browse the repository at this point in the history
Do not swallow execute errors
  • Loading branch information
helfer authored Mar 17, 2017
2 parents dd6eeff + b24542e commit 99cbb14
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions src/core/QueryManager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -753,10 +753,7 @@ export class QueryManager {
public startQuery<T>(queryId: string, options: WatchQueryOptions, listener: QueryListener) {
this.addQueryListener(queryId, listener);

this.fetchQuery<T>(queryId, options)
// `fetchQuery` returns a Promise. In case of a failure it should be caucht or else the
// console will show an `Uncaught (in promise)` message. Ignore the error for now.
.catch((error: Error) => undefined);
this.fetchQuery<T>(queryId, options);

return queryId;
}
Expand Down

0 comments on commit 99cbb14

Please sign in to comment.