Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Make Wallet first-class citizens #3784

Closed
ngotchac opened this issue Dec 9, 2016 · 0 comments · Fixed by #3990
Closed

Make Wallet first-class citizens #3784

ngotchac opened this issue Dec 9, 2016 · 0 comments · Fixed by #3990
Assignees
Labels
F8-enhancement 🎊 An additional feature request.

Comments

@ngotchac
Copy link
Contributor

ngotchac commented Dec 9, 2016

Make wallets for which at least one of the owner is in the accounts first-class citizens ; ie. being able to send transactions from then from anywhere in the app && keep their balances up-to-date (thus receiving notification on new transaction on an owned wallet).

@ngotchac ngotchac added F8-enhancement 🎊 An additional feature request. M7-ui labels Dec 9, 2016
@ngotchac ngotchac self-assigned this Dec 9, 2016
gavofyork pushed a commit that referenced this issue Dec 30, 2016
* Fixed hint in Address Select + Wallet as first-class-citizen

* Separate Owned and not Owned Wallets

* Fix balance not updating

* Fix MethodDecoding for Contract Deployment

* Fix TypedInput params

* Fix Token Transfer for Wallet

* Small change to contracts

* Fix wallets shown twice

* Fix separation of accounts and wallets in Accounts

* Fix linting

* Execute contract methods from Wallet ✓

* Fixing linting

* Wallet as first-class citizen: Part 1 (Manual) #3784

* Lower level wallet transaction convertion

* Fix linting

* Proper autoFocus on right Signer input

* PR Grumble: don't show Wallets in dApps Permissions

* Add postTransaction and gasEstimate wrapper methods

* Extract Wallet postTx and gasEstimate to utils + PATCH api

* Remove invalid test

It's totally valid for input's length not to be a multiple of 32 bytes. EG. for Wallet Contracts

* Merge master

* Fix linting

* Fix merge issue

* Rename Portal

* Rename Protal => Portal (typo)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
F8-enhancement 🎊 An additional feature request.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant