Add support for using auto fee calculation #22

Merged

nabarun merged 19 commits from deep-stack/registry-sdk:iv-add-tx-info-test into main 2024-09-06 09:07:58 +00:00

Conversation 0 Commits 19 Files Changed 14

+227 -49

prathamesh commented 2024-08-26 12:11:27 +00:00

Member

Copy Link

Copy Source

Part of Create a public laconicd testnet

• Add an option for setting gas price
• Use fees as gas estimation multiplier when gasPrice is provided (following cosmjs pattern)
  • Default gas estimation multiplier: 2
• Add a method to query bonds by owners and fix tests
• Add a test for inclusion of batch of txs sent at once in the same block

Part of [Create a public laconicd testnet](https://www.notion.so/Create-a-public-laconicd-testnet-896a11bdd8094eff8f1b49c0be0ca3b8) - Add an option for setting gas price - Use fees as gas estimation multiplier when `gasPrice` is provided (following `cosmjs` pattern) - Default gas estimation multiplier: `2` - Add a method to query bonds by owners and fix tests - Add a test for inclusion of batch of txs sent at once in the same block

prathamesh added 7 commits 2024-08-26 12:11:28 +00:00

Add test to send multiple txs cf5bb9e7de

Add test to check block height for all txs 0880185a4b

Refactor Batch txs test 6aad0e2506

Add option for setting gas price bc3010f67b

Accept fees optionally as gas estimation multiplier 74d5631753

Update package version 8a14027316

Set default gas estimation multiplier to 2 2222276c73

prathamesh referenced this pull request from cerc-io/laconic-registry-cli 2024-08-26 12:15:31 +00:00

Add a config option and arg to set gas price for `auto` fees calculation #81

prathamesh added 1 commit 2024-08-27 06:09:27 +00:00

Fix test for batched txs 06adca6e81

prathamesh added 1 commit 2024-08-27 09:00:51 +00:00

Add a method to query bonds by owner and update tests 9a1abc18ca

prathamesh added 1 commit 2024-08-27 09:45:54 +00:00

Add config tests 75e302fd29

prathamesh added 1 commit 2024-08-27 09:48:41 +00:00

Update test message fea30985c5

prathamesh added 1 commit 2024-08-27 10:46:28 +00:00

Reduce number of txs in batched txs test ee4c431bc8

prathamesh changed title from [WIP] Add support for using auto gas estimation to [WIP] Add support for using auto fee calculation 2024-08-27 10:59:33 +00:00

prathamesh changed title from [WIP] Add support for using auto fee calculation to [WIP] Add support for using `auto` fee calculation 2024-08-27 11:01:59 +00:00

prathamesh added 2 commits 2024-08-27 13:08:39 +00:00

Debug logs 1a42cf3836

Use gitea laconicd URL 5db0910a05

prathamesh force-pushed iv-add-tx-info-test from 5db0910a05 to 6add6d2844 2024-08-27 13:10:34 +00:00 Compare

prathamesh added 1 commit 2024-08-27 13:59:32 +00:00

Check all txs are within two blocks in batched txs test f802e1d31c

prathamesh added 1 commit 2024-09-05 05:51:27 +00:00

Update config tests to use a single account 8828b5388d

ashwin force-pushed iv-add-tx-info-test from 8828b5388d to cb1d74614c 2024-09-05 09:05:24 +00:00 Compare

Ghost added 1 commit 2024-09-05 12:01:23 +00:00

Update test to query bonds by owner d30245c6df

prathamesh force-pushed iv-add-tx-info-test from d30245c6df to 91a4792346 2024-09-05 13:51:40 +00:00 Compare

Ghost added 1 commit 2024-09-06 05:38:33 +00:00

Update bonds test adf39a4089

prathamesh marked the pull request as ready for review 2024-09-06 06:11:41 +00:00

prathamesh added 1 commit 2024-09-06 06:13:00 +00:00

Increment package version b56cb09b3b

prathamesh added 1 commit 2024-09-06 06:47:39 +00:00

Move chain id constant to test helpers 7717b11fb2

prathamesh added 1 commit 2024-09-06 06:54:56 +00:00

Rename method to query bonds by owners e88105f67b

nabarun merged commit 6a17b1b175 into main 2024-09-06 09:07:58 +00:00

nabarun deleted branch iv-add-tx-info-test 2024-09-06 09:08:00 +00:00

nabarun referenced this issue from a commit 2024-09-06 09:08:00 +00:00

Add support for using `auto` fee calculation (#22)

nabarun referenced this issue from a commit 2024-09-06 09:44:50 +00:00

Add a config option and arg to set gas price for `auto` fees calculation (#81)

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

Copied from Github

An issue or PR manually copied from GitHub.

Kind/Breaking

Breaking change that won't be backward compatible

Kind/Bug

Something is not working

Kind/Documentation

Documentation changes

Kind/Enhancement

Improve existing functionality

Kind/Feature

New functionality

Kind/Security

This is security issue

Kind/Testing

Issue or pull request related to testing

Priority

Critical

The priority is critical

Priority

High

The priority is high

Priority

Low

The priority is low

Priority

Medium

The priority is medium

Reviewed

Confirmed

Issue has been confirmed

Reviewed

Duplicate

This issue or pull request already exists

Reviewed

Invalid

Invalid issue

Reviewed

Won't Fix

This issue won't be fixed

Status

Abandoned

Somebody has started to work on this but abandoned work

Status

Blocked

Something is blocking this issue or pull request

Status

Need More Info

Feedback is required to reproduce issue or to continue work

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

AFDudley Alex cerccicd ericlewis i-norden (Ian Norden) prathamesh (Prathamesh Musale) rachmaninovquar rjk-laconic roysc (Roy Crihfield) srw zramsay

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: cerc-io/registry-sdk#22

Allow edits from maintainers