-
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
Ordinal transaction functions #201
Ordinal transaction functions #201
Conversation
|
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.
lgtm
@abdulhaseeb4239 can we merge this, since it's a non-breaking change? can do more testing on the web-extension and mobile PRs |
yes we can merge |
🔘 PR Type
What kind of change does this PR introduce?
📜 Background
We needed to move all the ordinals transactions logic to xverse-core. Including fetching and address UTXOs and getting the correct UTXO to send.
Issue Link: https://linear.app/xverseapp/issue/ENG-2528/move-ordinal-transactions-logic-to-xverse-core
Context Link (if applicable):
🔄 Changes
I have created new functions to wrap up the existing ones. I passed the ordinal inscription and
isRecover
boolean. and function fetches the addressUtxos from ordinalAddress and from btcAddress if it's a recover transaction.Changes:
Impact:
✅ Review checklist
Please ensure the following are true before merging: