Skip to content
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

Allow users to update the public keys for their existing accounts #57

Merged
merged 7 commits into from
May 16, 2019

Conversation

jcardenas9x
Copy link
Contributor

⚠️ - You will need to use check that your eosio-toppings is using the update-auth branch otherwise this branch will not be usable, or you can wait until that branch has been merged into develop

Changes

  • (3387) - Allow users to change the public key on their accounts, allowing them to also update their private key simultaneously
  • (3387) - Users can opt to change active permission's key pair, or owner permission's key pair, or both at the same time
  • (3386) - If user pushes updateauth in the Push Action page, the permission which got changed will have the private key reset. You will need to import the missing key in this case, but the permission which did not get updated will still be usable

🔎 Please note, importing private keys to accounts with no private keys in the local storage remains unchanged compared to previous updates

…e public keys for accounts that already have private keys assigned to them
…sion is set to undefined, the default permission is changed to the updated owner permission
… in 1 transaction when changing both keys, only 1 action when changing only 1 key
… import/edit account screen to the top of the viewport after clicking 'submit' so user can see the success/error prompt
@terrylks terrylks merged commit ab87acd into develop May 16, 2019
@terrylks terrylks deleted the modify-all-keys branch May 16, 2019 03:01
@jcardenas9x jcardenas9x added the enhancement New feature or request label May 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants