-
Notifications
You must be signed in to change notification settings - Fork 22
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
docs: add base64 example #355
Conversation
Signed-off-by: Michael Kantor <[email protected]>
Coverage report
Test suite run success115 tests passing in 11 suites. Report generated by 🧪jest coverage report action from e39ef06 |
demos/react-dapp/src/App.tsx
Outdated
<hr /> | ||
<h2>Pairing and session management:</h2> |
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.
I assume you want these removing as the button title for disconnecting the sessions is obvious enough?
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.
I assume you want these removing as the button title for disconnecting the sessions is obvious enough?
haha no, I removed this by accident. Thanks for flagging
Signed-off-by: Michael Kantor <[email protected]>
// Create multi-signature account | ||
const handleCreateMultisigAccount = async () => { | ||
// Fetch public keys from mirror node for each account | ||
const fetchPublicKey = async (accountId: string) => { | ||
const response = await fetch( | ||
`https://testnet.mirrornode.hedera.com/api/v1/accounts/${accountId}` | ||
`https://testnet.mirrornode.hedera.com/api/v1/accounts/${accountId}`, |
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.
don't need a comma here, is this part of the prettier config?
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.
Yup, that's because of prettier. It's also best practice to add trailing commas.
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
Description:
Adds a small tool to test executing a base64 transaction directly. This has been super helpful in testing the SaucerSwap integration.
Checklist