lavanet / lava

Apache License 2.0

407 stars 206 forks source link

fix: Unstake CLI fee grant fix #1648

Closed oren-lava closed 2 months ago

oren-lava commented 2 months ago

Description

Closes: #XXXX

Fixed a small bug in which a provider can't unstake using the CLI due to feegrant code.

Author Checklist

All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues.

I have...

[ ] read the contribution guide
[ ] included the correct type prefix in the PR title, you can find examples of the prefixes below:
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] targeted the main branch
[ ] provided a link to the relevant issue or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] included the necessary unit and integration tests
[ ] updated the relevant documentation or specification, including comments for documenting Go code
[ ] confirmed all CI checks have passed

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.

I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic, API design and naming, documentation is accurate, tests and test coverage

github-actions[bot] commented 2 months ago

Test Results

2 173 tests +125 2 173 :white_check_mark: +128 25m 52s :stopwatch: + 1m 32s 145 suites + 2 0 :zzz: ± 0 7 files + 1 0 :x: - 3

Results for commit 7d8cacd8. ± Comparison against base commit ea235a13.

:recycle: This comment has been updated with latest results.