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

Use default account for sending transactions #5588

Merged
merged 3 commits into from
May 16, 2017
Merged

Conversation

ngotchac
Copy link
Contributor

Closes #3135

In deploy/contract execution/create wallet => use the default account (set in dapps), but switch to the latest account used for those transaction if it changes

@ngotchac ngotchac added A0-pleasereview 🤓 Pull request needs code review. M7-ui labels May 10, 2017
@jacogr
Copy link
Contributor

jacogr commented May 16, 2017

Ok as a stop-gap, but not really future-proof where we move all views to dapps. (Where we set and use it is in different contexts)

@jacogr jacogr added A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging. and removed A0-pleasereview 🤓 Pull request needs code review. labels May 16, 2017
@gavofyork gavofyork merged commit 8642300 into master May 16, 2017
@gavofyork gavofyork deleted the ng-default-account branch May 16, 2017 14:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants