-
Notifications
You must be signed in to change notification settings - Fork 1
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
170 add update increment rewards option for staking #181
170 add update increment rewards option for staking #181
Conversation
✅ Deploy Preview for rococo-souffle-a625f5 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Looks good to me overall but I added some comments.
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.
Code looks good, can confirm that updating works now 👍
I've randomly added a title to each collator's cell with all the non-null information such as email/twitter/etc. I did this because the user deserves to know, but besides, bc the display name of some of them was a hexadecimal string. Taking into account that others are normally shown, I assumed in those cases there was an error on their end (i.e. when calling the Identity pallet), and the title helps identify the collator name better (see attached screenshot)
I didn't call the extrinsic on wallet connect, because I realize the extrinsic is called by the user, and he/she would have to sign, and pay fees. And it's simply horrible UX to do this without user action.
Closes Add Update / Increment rewards option for staking #170