Skip to content

Commit

Permalink
Merge pull request coinbase#96 from rmm5t/remove-nested-promise-loada…
Browse files Browse the repository at this point in the history
…llorders

Remove unnecessary nested promise in GDAX loadAllOrders()
  • Loading branch information
CjS77 authored Nov 28, 2017
2 parents c26e277 + c7454f1 commit 2b4a412
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions src/exchanges/gdax/GDAXExchangeAPI.ts
Original file line number Diff line number Diff line change
Expand Up @@ -274,11 +274,7 @@ export class GDAXExchangeAPI implements PublicExchangeAPI, AuthenticatedExchange
}
});
};
return new Promise((resolve, reject) => {
return loop(null).then((orders) => {
return resolve(orders);
}, reject);
});
return loop(null);
}

loadBalances(): Promise<Balances> {
Expand Down

0 comments on commit 2b4a412

Please sign in to comment.