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

Prettier GlobalFee fees errors #809

Merged
merged 4 commits into from
Aug 24, 2023
Merged

Prettier GlobalFee fees errors #809

merged 4 commits into from
Aug 24, 2023

Conversation

Reecepbcups
Copy link
Contributor

@Reecepbcups Reecepbcups commented Aug 24, 2023

I have seen a lot of developers confused by the CLI fee message (all of cosmos, not just Juno). This makes it prettier so it's easier to understand


not enough
image

not accepted
image

none input at all but one is required
image

@Reecepbcups Reecepbcups changed the title Prettier GlobalFee fees error Prettier GlobalFee fees errors Aug 24, 2023
@Reecepbcups Reecepbcups marked this pull request as ready for review August 24, 2023 19:25
@piscesyuma
Copy link

It's looks so pretty.

Copy link
Contributor

@JakeHartnell JakeHartnell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice improvement!

@Reecepbcups Reecepbcups merged commit 023a2ab into main Aug 24, 2023
19 checks passed
@Reecepbcups
Copy link
Contributor Author

How did commit 27301e6 get pushed to branch but not in the PR what

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants