fix: other network fees when higher than ARK static fees #1172
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1172 +/- ##
===========================================
+ Coverage 42.92% 42.96% +0.04%
===========================================
Files 217 217
Lines 5642 5646 +4
Branches 1075 1084 +9
===========================================
+ Hits 2422 2426 +4
+ Misses 3100 3099 -1
- Partials 120 121 +1
Continue to review full report at Codecov.
|
src/renderer/services/client.js
Outdated
@@ -669,9 +671,10 @@ export default class ClientService { | |||
* @returns {Object} | |||
*/ | |||
async buildTransfer ({ amount, fee, recipientId, vendorField, passphrase, secondPassphrase, wif }, isAdvancedFee = false, returnObject = false) { | |||
const staticFee = store.getters['transaction/staticFee'](0) || V1.fees[0] | |||
// To ensure that transfers cannot be build with a bigger fee than V1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// To ensure that transfers cannot be build with a bigger fee than V1 | |
// To ensure that transfers cannot be built with bigger fees than the static fees allowed by the peer or V1 |
or remove the comment if you think that is unnecessary
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will remove as it's only in the transfer method, thanks 👌
Your pull request needs some changes. Please wait for a comment from one of our developers for more information. |
* fix: improve validation for epoch format (#1164) * chore: upgrade Vue and other dependencies and use Node 11 on CI (#1160) * feat: allow using a pool of background workers and use create a unified HTTP client (#1120) * fix: `InputSelect` dropdowns (#1174) * fix: `InputSelect` dropdowns * refactor: use the new Vue 2.6 syntax for named slots * fix: do not allow more than 1 `InputSelect` open at the same time * fix: other network fees when higher than ARK default static fees (#1172) * fix: main sidebar menus (settings and peers/networks) (#1175) * fix: the wallet selection dropdown and the wallet sidebar filters (#1184) * fix: do not fail when navigating to other page while loading transactions (#1185) * fix: use of getStruct in ledger transactions (#1190) * chore: add ItsANameToo as code owner * fix: add network modal validation (#1173) * fix: network validation being overwritten * feat: pull active delegates from api * fix: make sure active delegates value exists * refactor: improve server regex * test: network modal for adding network * chore: replace todo with comment * fix: use one loop with prefilled values as backup * refactor: allow string or number for input text * refactor: allow fetching fees for specific network * fix: fetch network fees when adding/editing * fix: fetch fees & wait before add/update * chore: remove mario from contributors (#1196) * chore: add Altilly Exchange Wallet (#1199) * fix: align the add profile placeholder (#1202) * fix: days on the X axis of the market chart (#1203) * fix: fetch fees only if the network is available (#1206) * fix: stop hover from displacing address container (#1242) * fix: vue-i18n-extract commands (#1247) * refactor: require user action when generating 2nd passphrase (#1229) * feat: store sidebar sorting and filters (#1148) * feat: add option to hide the wallet button text (#1146) * fix: get children of TransitionGroup when available & word-break of vendorfield (#1234) * feat: show remaining bytes in vendorfield helper text (#1259) * chore: use organization-wide GitHub Configuration (#1267) * misc: Italian language update (#1268) * fix: tidy permission method names & check exists (#1260) * feat: add option to export wallets (#1244) * feat: use the theme colours for the wallet filters instead of white (#1270) * deps: upgrade `axios` (CVE-2019-10742) (#1271) * fix: saving of invalid profile name on leave (#1101) * bugfix: saving of invalid profile editions * refactor: dont disable save button when profile name valid even when the 'isNameEditable' don't disable save button. Instead disable it only when the name has an error. * Update en-US.js * Update ProfileEdition.vue * refactor: show failed update message on close * refactor: add update failure reason * fix: disable both save buttons if name invalid * fix: display the wallet sidebar filters on the right position (#1201) * fix: highlight filters button when expanded if active * fix: display the wallet sidebar filters on the right position * refactor: add the status icon and plus/minus sign in the transaction show modal (#1272) * fix: use webview instead of iframe for changelly (#1277) * revert: worker changes from commit e42290 * refactor: use nock to mock tests (#1291) * fix: add options mock requests * fix: tests not mocking api * fix: duplicated key warning on `WalletNew` * Update src/renderer/pages/Wallet/WalletNew.vue Co-Authored-By: Edgar Goetzendorff <hello@dated.fun> * fix: ledger options on build (#1300) * fix: ledger options on build * fix: use component directly * fix: change order of arguments in uniqBy call (#1297) * fix: don't close custom peer modal when clicking inside (#1204) * refactor: rename `NetworkCustomPeer` to `NetworkCustomPeerModal` * fix: do not close the custom peer modal when clicking inside it * refactor: wallet address & balance clickable on WalletAll page (#1301) * fix: ledger wallet sidebar filter (#1292) * fix: ledger whitescreen (#1296) * fix: ledger whitescreen errors * refactor: all ledger errors are failures * refactor: improve ledger connection check * refactor: improve ledger & use hid-singleton * test: fix failing * chore: remove e2e tests * fix: open transaction modal on dashboard (#1281) * fix: remove deeplink validation prefix * fix: open transaction modal on dashboard * fix: don't switch twice when selecting wallet on WalletSidebar (#1302) * chore: bump version to 2.5.0 (#1303)
Proposed changes
Resolves an issue with non-ARK chains, where the fee greater than ARK's static fees resulted in not being able to send transactions.
Types of changes
Checklist