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

XCM. Fee verification needs to be updated #2833

Open
stepanLav opened this issue Dec 11, 2024 · 0 comments · May be fixed by #2856
Open

XCM. Fee verification needs to be updated #2833

stepanLav opened this issue Dec 11, 2024 · 0 comments · May be fixed by #2856
Assignees
@pgolovkin
Labels
Bug Issues & PRs that addressing a bug Clarification required Issues require more info or research Design required Issues require UX/UI design before development

Comments

@stepanLav
Copy link
Member

Description

Currently we do not check cross-chain fee in Available balance when make XCMs

Expected result:
Show error - "Insufficient balance to pay cross-chain fee"
Would be better to make it more responsive and show which exact token insufficiently and on which account

Environment

e94c642
@stepanLav stepanLav added the Bug Issues & PRs that addressing a bug label Dec 11, 2024
@stepanLav stepanLav added Clarification required Issues require more info or research Design required Issues require UX/UI design before development labels Dec 11, 2024
@stepanLav stepanLav mentioned this issue Dec 11, 2024
Closed
@johnthecat johnthecat moved this from Todo to In Review in Nova Spektr development Dec 12, 2024
@johnthecat johnthecat linked a pull request Dec 12, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pgolovkin pgolovkin

Labels
Bug Issues & PRs that addressing a bug Clarification required Issues require more info or research Design required Issues require UX/UI design before development
Projects
Nova Spektr development
Status: In Review
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: fee calculation for foreign assets novasamatech/nova-spektr
2 participants
@pgolovkin @stepanLav