-
Notifications
You must be signed in to change notification settings - Fork 7
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
[PLA-1840] add account update for sign #131
Conversation
PR Reviewer Guide 🔍
|
PR Code Suggestions ✨
|
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.
updates
export default `mutation UpdateUser($email: String, $password: String) { | ||
UpdateUser(email: $email, password: $password) | ||
export default `mutation UpdateUser($email: String, $password: String, $account: String) { | ||
UpdateUser(email: $email, password: $password, account: $account) |
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.
This attribute is dedicated for wallet daemon and we cannot update this. I created a new attribute for this in this PR https://github.com/enjin/platform-multi-tenant/pull/72
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.
okay thank you! will check the new implementation
update made with another ticket #135 |
PR Type
Enhancement, Bug fix
Description
SignTransaction.vue
to include logic for updating user wallet account if the application is multi-tenant.updateUserAccount
method toAuthApi
class inauth.ts
.UpdateUser
GraphQL mutation to accept anaccount
variable.updateUserAccount
method touseAppStore
inindex.ts
.Changes walkthrough 📝
SignTransaction.vue
Update user wallet account in multi-tenant setup
resources/js/components/SignTransaction.vue
publicKeyToAddress
from~/util/address
.auth.ts
Add updateUserAccount method in AuthApi
resources/js/api/auth.ts
updateUserAccount
method toAuthApi
class.UpdateUser.ts
Extend UpdateUser mutation to include account variable
resources/js/graphql/mutation/auth/UpdateUser.ts
UpdateUser
mutation to includeaccount
variable.index.ts
Add updateUserAccount method in useAppStore
resources/js/store/index.ts
updateUserAccount
method touseAppStore
.