Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Compute ccuFee property of chain connector plugin and use a safe value by default #9147

Closed
Tracked by #9158
ishantiw opened this issue Nov 22, 2023 · 0 comments · Fixed by #9153
Closed
Tracked by #9158

Compute ccuFee property of chain connector plugin and use a safe value by default #9147

ishantiw opened this issue Nov 22, 2023 · 0 comments · Fixed by #9153
Assignees
@Phanco
Labels
framework-plugins/chain_connector type: bug
Milestone
Sprint 109

Comments

@ishantiw
Copy link
Contributor

Description

Currently, minFee part of the chain connector plugin has a default value of 10LSK which is a huge amount for transaction.
There are couple of improvments:

  • Lower the default value to a lower safe value 500000 beddows
  • Use receivingChainAPIClient.transaction.computeMinFee function to compute minFee on the fly and submit CCU transaction

Which version(s) does this affect? (Environment, OS, etc...)

6.0.0
@Phanco Phanco self-assigned this Nov 22, 2023
@Madhulearn Madhulearn added this to the Sprint 109 milestone Nov 22, 2023
@Madhulearn Madhulearn moved this to In Progress in Lisk SDK Version 6.0.0 Nov 23, 2023
@Madhulearn Madhulearn moved this to In Progress in Lisk SDK Version 6.1.0 Nov 23, 2023
@Phanco Phanco mentioned this issue Nov 24, 2023
Merged
@github-project-automation github-project-automation bot moved this from In Progress to Done in Lisk SDK Version 6.1.0 Nov 24, 2023
@ishantiw ishantiw changed the title Update ccuFee property of chain connector plugin to use a safe value by default Compute ccuFee property of chain connector plugin and use a safe value by default Nov 24, 2023
@Madhulearn Madhulearn mentioned this issue Nov 27, 2023
Closed
5 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Phanco Phanco

Labels
framework-plugins/chain_connector type: bug
Projects
None yet
Milestone
Sprint 109
Development

Successfully merging a pull request may close this issue.

Update Default CCU Fee LiskArchive/lisk-sdk
3 participants
@ishantiw @Phanco @Madhulearn