Fixed bug that made send screen sometimes transition to account detail

feature/default_network_editable
Dan Finlay 8 years ago
parent 45138af6c6
commit 2e16e1eb94
  1. 1
      app/scripts/lib/tx-utils.js
  2. 3
      ui/app/conf-tx.js
  3. 2
      ui/app/reducers/app.js

@ -106,6 +106,7 @@ module.exports = class txProviderUtils {
txParams.gasLimit = normalize(txParams.gasLimit || txParams.gas) txParams.gasLimit = normalize(txParams.gasLimit || txParams.gas)
txParams.nonce = normalize(txParams.nonce) txParams.nonce = normalize(txParams.nonce)
// build ethTx // build ethTx
log.info(`Prepared tx for signing: ${JSON.stringify(txParams)}`)
const ethTx = new Transaction(txParams) const ethTx = new Transaction(txParams)
return ethTx return ethTx
} }

@ -13,6 +13,7 @@ const BN = ethUtil.BN
const PendingTx = require('./components/pending-tx') const PendingTx = require('./components/pending-tx')
const PendingMsg = require('./components/pending-msg') const PendingMsg = require('./components/pending-msg')
const PendingPersonalMsg = require('./components/pending-personal-msg') const PendingPersonalMsg = require('./components/pending-personal-msg')
const Loading = require('./components/loading')
module.exports = connect(mapStateToProps)(ConfirmTxScreen) module.exports = connect(mapStateToProps)(ConfirmTxScreen)
@ -48,7 +49,7 @@ ConfirmTxScreen.prototype.render = function () {
var isNotification = isPopupOrNotification() === 'notification' var isNotification = isPopupOrNotification() === 'notification'
log.info(`rendering a combined ${unconfTxList.length} unconf msg & txs`) log.info(`rendering a combined ${unconfTxList.length} unconf msg & txs`)
if (unconfTxList.length === 0) return null if (unconfTxList.length === 0) return h(Loading)
return ( return (

@ -291,7 +291,7 @@ function reduceApp (state, action) {
case actions.SHOW_CONF_TX_PAGE: case actions.SHOW_CONF_TX_PAGE:
return extend(appState, { return extend(appState, {
currentView: { currentView: {
name: pendingTxs ? 'confTx' : 'account-detail', name: 'confTx',
context: 0, context: 0,
}, },
transForward: action.transForward, transForward: action.transForward,

Loading…
Cancel
Save