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

Namadillo: Gas amount selection modal #788

Open
pedrorezende opened this issue May 29, 2024 · 4 comments · May be fixed by #832
Open

Namadillo: Gas amount selection modal #788

pedrorezende opened this issue May 29, 2024 · 4 comments · May be fixed by #832
Assignees
Milestone

Comments

@pedrorezende
Copy link
Collaborator

This modal should enable users to specify the amount of gas they wish to spend on their transactions, taking the minimum transaction fee into account for validation purposes.

Additionally, we need to establish a method for calculating the range that defines the 'Average' and 'High' gas amount settings.

Please refer to the image below for a visual representation of the expected interface

image
@pedrorezende pedrorezende added this to the PoS, Governance milestone May 29, 2024
@pedrorezende pedrorezende self-assigned this May 29, 2024
@pedrorezende pedrorezende linked a pull request Jun 2, 2024 that will close this issue
@jurevans
Copy link
Collaborator

Closing for now - if we want to revisit this issue and get this feature in, we will reopen

@pedrorezende
Copy link
Collaborator Author

Reopening the issue in order to give users the ability of increasing gas amount

@brentstone
Copy link

@Fraccaman @grarco I think this is important to have in Namadillo soon. Do we have all info in the indexer needed to accomplish this? Also tagging @jurevans and @mateuszjasiuk

@euharrison
Copy link
Collaborator

curl -X 'GET' \
  'http://localhost:5001/api/v1/gas/estimate?shielded_transfer=1' \
  -H 'accept: application/json'

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

Successfully merging a pull request may close this issue.

4 participants