-
Notifications
You must be signed in to change notification settings - Fork 208
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
Closed
Closed
Changes from 23 commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
1df5e01
feat(wallet-ui): much reworking of wallet connections
michaelfig 7513d95
feat: pretty up the nonfunctional SmartWalletConnection
michaelfig c9d0038
inline classname
turadg 0cc60af
enumerate ConnectionStatus
turadg e1e597d
refactor
turadg af95fcf
localStorage helpers
turadg 0bb93d6
chore: move `Application.jsx` to `Provider.jsx`
michaelfig dd9466a
fix: use new `Application.jsx`
michaelfig fd407e2
KLUDGE: avoid AttestationTool import for now
dckc af48aec
chore: @agoric/smart-wallet package scaffold
dckc 2b8d3d9
chore: prune vat-walletManager
dckc 6a3c93f
feat: home.smartWallet
dckc cf0c146
build: fix smart-wallet contract deps
dckc 70a7741
feat: connect smart-wallet with chainStorage
dckc 6f7b0fc
chore: combine parts of smart wallet subscription
dckc f056824
feat(wallet-ui): merge ChainConnector into SmartWalletConnection
michaelfig 2db3d8a
fix(smart-wallet): fresh initial state
michaelfig f3436f1
wip: follow casting from wallet ui
michaelfig b19bdd9
chore: update package names
michaelfig cb3f14b
fix(casting): properly unmarshal presences
michaelfig b302e41
feat(wallet-ui): read-only access to on-chain wallet
michaelfig ffaf649
fix(casting): more error handling
michaelfig 36bdd7b
fix(wallet-ui): ready for demo
michaelfig ac8a288
makeSubscriptionKit no longer takes a state
turadg b137ad3
test: start of smart-wallet contract test
turadg File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file was deleted.
Oops, something went wrong.
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -35,7 +35,6 @@ import { makePaymentActions } from './actions.js'; | |
|
||
import '@agoric/store/exported.js'; | ||
import '@agoric/zoe/exported.js'; | ||
import '@agoric/run-protocol/exported.js'; | ||
|
||
import './internal-types.js'; | ||
import './types.js'; | ||
|
@@ -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 commentThe reason will be displayed to describe this comment to others. Learn more. before landing or after? |
||
/** @type import('@endo/promise-kit').PromiseKit<AttestationTool> */ | ||
const attMakerPK = makePromiseKit(); | ||
|
||
|
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
// This file can contain .js-specific Typescript compiler config. | ||
{ | ||
"compilerOptions": { | ||
"target": "esnext", | ||
"module": "esnext", | ||
|
||
"noEmit": true, | ||
/* | ||
// The following flags are for creating .d.ts files: | ||
"noEmit": false, | ||
"declaration": true, | ||
"emitDeclarationOnly": true, | ||
*/ | ||
"downlevelIteration": true, | ||
"strictNullChecks": true, | ||
"moduleResolution": "node", | ||
}, | ||
"include": [ | ||
"*.js", | ||
"scripts/**/*.js", | ||
"src/**/*.js", | ||
"test/**/*.js", | ||
"tools/**/*.js", | ||
], | ||
} |
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,45 @@ | ||
{ | ||
"name": "@agoric/smart-wallet", | ||
"version": "0.1.1", | ||
"description": "Wallet contract", | ||
"type": "module", | ||
"scripts": { | ||
"build": "exit 0", | ||
"test": "ava", | ||
"test:xs": "exit 0", | ||
"lint": "run-s --continue-on-error lint:*", | ||
"lint-fix": "yarn lint:eslint --fix", | ||
"lint:types": "tsc --maxNodeModuleJsDepth 3 -p jsconfig.json", | ||
"lint:eslint": "eslint ." | ||
}, | ||
"devDependencies": { | ||
"ava": "^3.12.1" | ||
}, | ||
"dependencies": { | ||
"@agoric/wallet-backend": "0.12.1", | ||
"@agoric/deploy-script-support": "^0.9.0", | ||
"@agoric/zoe": "^0.24.0", | ||
"@agoric/notifier": "^0.4.0", | ||
"@endo/far": "^0.2.3" | ||
}, | ||
"keywords": [], | ||
"repository": { | ||
"type": "git", | ||
"url": "git+https://github.com/Agoric/agoric" | ||
}, | ||
"author": "Agoric", | ||
"license": "Apache-2.0", | ||
"bugs": { | ||
"url": "https://github.com/Agoric/agoric/issues" | ||
}, | ||
"homepage": "https://github.com/Agoric/agoric#readme", | ||
"ava": { | ||
"files": [ | ||
"test/**/test-*.js" | ||
], | ||
"timeout": "2m" | ||
}, | ||
"publishConfig": { | ||
"access": "public" | ||
} | ||
} |
Oops, something went wrong.
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.
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 anag-solo
usemyAddressNameAdmin
to set updepositFacet
. There can be only one. This is why we see 0 invitations in xnet #5513 (comment) .