-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Observer Mode #17792
Observer Mode #17792
Conversation
9d1cb4b
to
6a05d23
Compare
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.
aok
This is in draft because it probably should not go in while there is an outstanding release branch. |
|
This is probably the reason for the coverage issue |
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.
aok
This PR is the first "catchup" into main for a feature from the long_lived/vault branch: observer mode.
With these changes, it will be possible to add just a pubkey to the keyring and wallets will sync balances as usual. What this does not add is the ability to sign transactions externally with the Chia Wallet, although the hooks to do so are included so in theory external signers could begin to be developed (pending confirmation of CHIPs 27-31). The Chia Wallet as a signer will likely be the next stage of "catchup"s from the long lived branch.
Changes from the following PRs are included in this diff:
@clvm_streamable
#17715)@clvm_streamable
#17715Additional changes after the initial commit b79e2ea may be made due to review comments