-
Notifications
You must be signed in to change notification settings - Fork 206
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
Implement smart-wallet and read-only UI #5594
Conversation
- add smartWallet bundle, installation to @agoric/vats - add .smartWallet property in makeClientBanks - initial smart-wallet.js contract
demo: $ agd query vstorage keys published.wallet children: - agoric1n9ea2m4987r93q4df8cckyg6n3sw685jx0jf8h $ agoric follow :published.wallet.agoric1n9ea2m4987r93q4df8cckyg6n3sw685jx0jf8h ...
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.
found a critical problem
const myAddressNameAdmin = E(namesByAddressAdmin).lookupAdmin(address); | ||
const myWallet = E(zoe).startInstance( | ||
smartWallet, | ||
{}, | ||
{ agoricNames, bank, namesByAddress, myAddressNameAdmin, board }, |
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.
Both the smartWallet
contract an ag-solo
use myAddressNameAdmin
to set up depositFacet
. There can be only one. This is why we see 0 invitations in xnet #5513 (comment) .
* marshaller?: ERef<Marshaller>, | ||
* }} privateArgs | ||
*/ | ||
export const start = async (zcf, privateArgs) => { |
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.
we need some test of this contract starting and the shape of its creatorFacet
. I can help with that if you like.
@@ -582,6 +581,7 @@ export function makeWallet({ | |||
|
|||
const getAttBrand = makeMemoizedGetter(() => E(getAttIssuer()).getBrand()); | |||
|
|||
/** @typedef {any} AttestationTool */ /* @@@ TODO */ |
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.
before landing or after?
overtaken by #5701 etc. |
closes: #5356
Description
Here's a (rather unimpressive) working demonstration of the smart wallet UI. Notice the zero-latency initialization of the view, and that it's currently read-only.
Screen.Recording.2022-06-14.at.3.11.50.PM.mov
Security Considerations
Documentation Considerations
Testing Considerations