-
Notifications
You must be signed in to change notification settings - Fork 19
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
[ENG-3474] feat: Make UI for STX RBF in the extension #727
[ENG-3474] feat: Make UI for STX RBF in the extension #727
Conversation
…-for-rbf-in-the-extension
…-for-rbf-in-the-extension
…-for-rbf-in-the-extension
* fix: cap stx contract call fees * chore: update core version * chore: bump to core 6.0.1
stxToMicrostacks, | ||
StxTransactionData, | ||
} from '@secretkeylabs/xverse-core'; | ||
import { deserializeTransaction, estimateTransaction } from '@stacks/transactions'; | ||
import { isLedgerAccount, microStxToStx } from '@utils/helper'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: xverse-core already exports stxToMicrostacks and microstacksToStx
stxToMicrostacks, | |
StxTransactionData, | |
} from '@secretkeylabs/xverse-core'; | |
import { deserializeTransaction, estimateTransaction } from '@stacks/transactions'; | |
import { isLedgerAccount, microStxToStx } from '@utils/helper'; | |
stxToMicrostacks, | |
microstacksToStx | |
StxTransactionData, | |
} from '@secretkeylabs/xverse-core'; | |
import { deserializeTransaction, estimateTransaction } from '@stacks/transactions'; | |
import { isLedgerAccount } from '@utils/helper'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The downside here is that microstacksToStx
requires a BigNumber
-only param, but of course we can convert it
@dhriaznov thanks for making the code improvements. let's merge this after the base branch is merged |
…on into denys/eng-3474-make-ui-for-stx-rbf-in-the-extension
Thanks for the review @teebszet! I resolved the git conflicts and applied the above code suggestions |
@dhriaznov needs conflicts resolved |
…on into denys/eng-3474-make-ui-for-stx-rbf-in-the-extension
Test with build here: https://github.com/secretkeylabs/xverse-web-extension/actions/runs/7489716984 |
🔘 PR Type
What kind of change does this PR introduce?
📜 Background
This feature is needed to help our Stacks users increase the chance of getting their pending transactions included in the next Stacks anchor block, by increasing the transaction fee.
Issue Link: #[ENG-3474]
Context Link (if applicable):
🔄 Changes
useRbfTransactionData
hooksrc/app/components/transactions/stxTransferTransaction.tsx
componentsrc/app/components/transactions/stxTransaction.tsx
componentsrc/assets/img/transactions/increaseFee.svg
iconImpact:
🖼 Screenshot / 📹 Video
The demo videos are in the linear ticket.
✅ Review checklist
Please ensure the following are true before merging: