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.

Sign up for GitHub

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

Jump to bottom

Enforce min-gas-price values appropriate for provenance network #310

Closed
4 tasks
iramiller opened this issue May 10, 2021 · 1 comment · Fixed by #324
Closed
4 tasks

Enforce min-gas-price values appropriate for provenance network #310

iramiller opened this issue May 10, 2021 · 1 comment · Fixed by #324
Assignees
@iramiller
Labels
bug Something isn't working CLI Command line interface feature

Comments

@iramiller
Copy link
Member

Summary

Currently the min-gas-price flag has no Provenance network specific validation

Proposal

Enforce a minimum gas price that is greater than zero and denominated in nhash only.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@iramiller iramiller added bug Something isn't working CLI Command line interface feature labels May 10, 2021
@iramiller
Copy link
Member Author

Along with this change we should set the default --min-gas-price=1905nhash which is up from the current value of 0.025nhash

@iramiller iramiller self-assigned this May 20, 2021
@iramiller iramiller mentioned this issue May 20, 2021
Merged
8 tasks
@iramiller iramiller moved this from Todo to Done in Provenance Core Protocol Team Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@iramiller iramiller

Labels
bug Something isn't working CLI Command line interface feature
Projects
Provenance Core Protocol Team
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Minimum gas price validation for mainnet provenance-io/provenance
1 participant
@iramiller