Skip to content
This repository has been archived by the owner on May 31, 2021. It is now read-only.

Gas limit setting to default 200000 is too low in some cases #161

Open
kwunyeung opened this issue Oct 25, 2020 · 0 comments
Open

Gas limit setting to default 200000 is too low in some cases #161

kwunyeung opened this issue Oct 25, 2020 · 0 comments
Assignees
Labels
kind/bug Something isn't working
Milestone

Comments

@kwunyeung
Copy link
Contributor

Generic information

  • OS name and version: iOS 14
  • Application build code: 5400
  • Network type (mobile/Wi-Fi): Wifi

Bug description

Transaction will be failed due to out of gas when there are many messages in a single transaction.

https://morpheus.desmos.network/transactions/D37BECE1DE2079EA25C1A5E045471069634B4735156D45466D383C792D2F5C4F

image

Steps to reproduce

Add a few posts or reactions within 30 seconds before the transaction is broadcasted. The transaction will be failed due to out of gas.

Expected behavior

The gas limit should be increased with the number of messages in the transaction is increased. We can consider getting the gas estimate with gas adjustment or simply add a fix number gas per message before we have the fixed minimum fee fully functional.

@RiccardoM RiccardoM added the kind/bug Something isn't working label Oct 27, 2020
@RiccardoM RiccardoM added this to the Version 0.6.0 milestone Oct 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants