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

feat: add transaction fee to claiming modal #1673

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

neocybereth
Copy link

@neocybereth neocybereth commented Feb 19, 2025

The change in action:

Screenshot 2025-02-19 at 1 23 45 PM

Gas Limit change properly reflected in the extension as well:

Screenshot 2025-02-19 at 1 07 36 PM

Closes #1645

@euharrison euharrison requested review from euharrison and pedrorezende and removed request for euharrison February 19, 2025 13:37
@euharrison
Copy link
Collaborator

@neocybereth the test is failing on StakingRewards.test.tsx

Do you have permission to see the test result?
https://github.com/anoma/namada-interface/actions/runs/13402745803/job/37491896481?pr=1673

Do you mind running the test locally to check what is going on?

Screenshot 2025-02-20 at 12 32 13

@neocybereth
Copy link
Author

alright @euharrison I pushed a fix to the tests

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.

Add the transaction fee modal to Claim Rewards
2 participants