https://github.com/firebase/snippets-web/blob/26569c25020d706a2f5a3b9bc2d306c29a7e5bfc/snippets/firestore-next/test-firestore/transaction_promise.js#L21 Should there be a `return newPop` in there before the else statement?