Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug: tx flags for txfees update prop #1546

Merged
merged 2 commits into from
May 19, 2022
Merged

bug: tx flags for txfees update prop #1546

merged 2 commits into from
May 19, 2022

Conversation

czarcas7ic
Copy link
Member

Can't submit txfees prop without fees flags

@czarcas7ic czarcas7ic requested a review from a team May 19, 2022 18:40
@czarcas7ic czarcas7ic added A:backport/v9.x Do not use. backport patches to v9.x branch A:backport/v8.x backport patches to v8.x branch and removed C:CLI C:x/txfees labels May 19, 2022
@p0mvn
Copy link
Member

p0mvn commented May 19, 2022

@czarcas7ic Was it not working before and just not used? Just curious how this was never caught

@codecov-commenter
Copy link

Codecov Report

Merging #1546 (f8c9898) into main (1877474) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1546   +/-   ##
=======================================
  Coverage   19.46%   19.46%           
=======================================
  Files         241      241           
  Lines       32183    32183           
=======================================
  Hits         6264     6264           
  Misses      24767    24767           
  Partials     1152     1152           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1877474...f8c9898. Read the comment docs.

@czarcas7ic
Copy link
Member Author

My guess is all the current submissions were done via lcd @p0mvn

@mergify mergify bot merged commit ae19a4c into main May 19, 2022
@mergify mergify bot deleted the czarcas7ic-patch-1 branch May 19, 2022 19:11
mergify bot pushed a commit that referenced this pull request May 19, 2022
Can't submit txfees prop without fees flags

(cherry picked from commit ae19a4c)
mergify bot pushed a commit that referenced this pull request May 19, 2022
Can't submit txfees prop without fees flags

(cherry picked from commit ae19a4c)
czarcas7ic added a commit that referenced this pull request May 19, 2022
Can't submit txfees prop without fees flags

(cherry picked from commit ae19a4c)

Co-authored-by: Adam Tucker <adamleetucker@outlook.com>
czarcas7ic added a commit that referenced this pull request May 19, 2022
Can't submit txfees prop without fees flags

(cherry picked from commit ae19a4c)

Co-authored-by: Adam Tucker <adamleetucker@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A:automerge A:backport/v8.x backport patches to v8.x branch A:backport/v9.x Do not use. backport patches to v9.x branch C:CLI C:x/txfees
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants