Add to react-app example of signing with wallet and then server signer #1059
+432
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Member
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
BundleMonUnchanged files (133)
No change in files bundle size Final result: ✅ View report in BundleMon website ➡️ |
mcintyre94
force-pushed
the
react-app-partial
branch
from
3e3bed0 to
441fafe
Compare
mcintyre94
changed the title
Contributor
|
Documentation Preview: https://kit-docs-260rp5duw-anza-tech.vercel.app |
mcintyre94
force-pushed
the
react-app-partial
branch
from
441fafe to
38db218
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem
I've seen a few people asking how to do this, and signers provide a really neat approach. Since I think it makes most sense to demonstrate this using a browser wallet, I've added it to the react-app example
Summary of Changes
The new "Partial Sign Transaction" panel is very similar to the existing "Sign Transaction" one.
A
mockApiRequesttakes a serializedTransactionas input, and returns theSignatureBytesfor this address (the keypair is stored in a JSON file). This mocks a server being used to sign transactions, for example as a fee payer.A custom
TransactionPartialSignercalledfeePayerSignersigns transactions by encoding them and passing them tomockApiRequest, mocking making an API request to a server that can sign a transaction.The connected wallet continues to be a signer, for the SOL transfer instruction.
When the transaction is signed, it is first sent to the user's wallet to be signed, and then it is signed by the custom
feePayerSigner. This ordering is because the user's wallet is a modifying signer, and is enforced bymockApiRequestusingsignTransactionwhich means that after it signs the transaction is fully signed.Fixes #1034