Skip to content

Commit

Permalink
[firestore] added missing conditional for firestore transaction error…
Browse files Browse the repository at this point in the history
…s - should now error on failed promise validations
  • Loading branch information
Salakar committed Apr 16, 2018
1 parent 5589ed7 commit b534e02
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/modules/firestore/TransactionHandler.js
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,7 @@ export default class TransactionHandler {
// reject the final promise and remove from native
// update is failed when either the users updateFunction
// throws an error or rejects a promise
if (updateFailed) {
if (updateFailed || finalError) {
// $FlowExpectedError: Reject will always be present
return reject(finalError);
}
Expand Down

0 comments on commit b534e02

Please sign in to comment.