-
Notifications
You must be signed in to change notification settings - Fork 3
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
[FEATURE] adds blockaid scan tx service method and route #121
Conversation
Please don't merge yet, we need to touch base with blockaid for failing API requests. |
Preview is available here: |
const NetworkNameBlockaid: { [index: string]: "pubnet" | "futurenet" } = { | ||
PUBLIC: "pubnet", | ||
FUTURENET: "futurenet", | ||
}; |
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.
Should this include testnet? From their docs, it looked like they support testnet now
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.
from their docs it does look that way but their type here is "pubnet" | "futurenet"
so I think maybe that's an open question to bring up with them.
Preview is available here: |
Something went wrong with PR preview build please check |
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@blockaid/[email protected] |
Preview is available here: |
What
Adds blockaid tx scanning and validation
Why
To be used in Freighter when a user signs a tx
Docs: https://docs.blockaid.io/reference/scan_transaction_scan_transaction_post