Joystream / pioneer

Governance app for Joystream DAO
https://pioneerapp.xyz/
GNU General Public License v3.0
44 stars 70 forks source link

Create validator profile #4330

Open dmtrjsg opened 1 year ago

dmtrjsg commented 1 year ago

Context

Design

🆕 Link to Figma>

Design Preview

Screenshot 2023-10-26 at 16 23 29

Scope

Add section to create membership modal for validators.

⚠️ When validator account added, the membership creation tx has additional step - binding account to membership. This happens before Membership is created/ updated (as a first step in the tx)

Screenshot 2023-10-26 at 16 23 36

⚠️ Pls make sure that all form inputs validation are done in accordance to the full scnearios of user story

ℹ️ Verification is covered in this story:

dmtrjsg commented 1 year ago

@chrlschwb designs are finalised

eshark9312 commented 1 year ago

@dmtrjsg Binding validator account tx can't be done before creating a membership.

As @thesan mentioned here, binding validator account with the membership is the same as binding any staking account, and there is no specific parameter for the validator staking account to differentiate it from other staking accounts which are used for other activities like Add New Proposal. Same for nominators.

I suggest to change the design.

dmtrjsg commented 1 year ago

Hey @eshark9312 thank you for raising this!

1️⃣ Why cannot we add validator accounts to bind in the same flow as creating membership? but just sign "creating" tx first in the multi-step of the tx modal? So we would have 3 steps when membership is created and same for updated?

2️⃣ For a more effective management of all bound accounts, I really like your idea, but this could be added later, right? As an improvement.

Let me know if I am missing smth.

cc @thesan

We will discuss and I will come back to you with details.

eshark9312 commented 1 year ago

@dmtrjsg

1️⃣ Why cannot we add validator accounts to bind in the same flow as creating membership? but just sign "creating" tx first in the multi-step of the tx modal? So we would have 3 steps when membership is created and same for updated?

-It can be done, but needs three step tx. -creating tx should be first. -From the current design, a member can add validator account by update membership modal, but he can add nominator account in validator/nominator page even though they are equal operation. That's the point.

dmtrjsg commented 1 year ago

@eshark9312 Thanks for the clarification, pls see the responses below.

Validator acc binding: step 1 - first part of binding step 2 - is to confirm but can be batched with account creation of membership

Nominator: Here the account is not actually bound with the account Add nominator account - any account can be added (no signature is required) Create stash account - aka set up new bonding preferences - this does not have to be signed with the membership controller account so case is slightly different to binding the validator acc (we do not strictly require to view members in context of nominators, but if controller account is the same for nominator stash and membership - then we surely can!)