Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

Apply the Fee Type from Settings in Purchase #540

Closed
jjeffryes opened this issue Jun 29, 2017 · 1 comment
Closed

Apply the Fee Type from Settings in Purchase #540

jjeffryes opened this issue Jun 29, 2017 · 1 comment

Comments

@jjeffryes
Copy link
Contributor

Settings/Advanced has a fee type which is saved to local storage, that type needs to be applied to the purchase.

Change the language to reflect the default, which is high, instead of suggesting average.

Consider showing the current estimated fee for each if possible.

@rmisio
Copy link
Contributor

rmisio commented Jun 29, 2017

Settings/Advanced has a fee type which is saved to local storage, that type needs to be applied to the purchase.

If we update the spend() function in the wallet/Spend.js model as follows, then it would use the local storage fee type for any wallet spends (i.e. both from the Wallet and from Purchasing):

  const attrs = {
    currency: app && app.settings && app.settings.get('localCurrency') || 'BTC',
    feeLevel: app && app.localSettings && app.localSettings.get('defaultTransactionFee') || 'high', 
    memo: '', // this could be removed
    ...fields,
  };

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants