search

Joystream / joystream

Joystream Monorepo
http://www.joystream.org
GNU General Public License v3.0
1.42k stars 115 forks source link

add CLI command `verifyValidator` #4933

Closed chrlschwb closed 9 months ago

chrlschwb commented 1 year ago
zeeshanakram3 commented 11 months ago

Shouldn't the workerRemak call be passed an encoded protobuf RemarkMetadataAction message with the VerifyValidator action set?

Correct. @chrlschwb please address that.

chrlschwb commented 11 months ago

Shouldn't the workerRemak call be passed an encoded protobuf RemarkMetadataAction message with the VerifyValidator action set?

Correct. @chrlschwb please address that.

Please let me know which is correct.

zeeshanakram3 commented 11 months ago

The user inputs member_id and is_verified, then the code encodes it to be used in the workerRemark extrinsic

This. For reference, please look at the implementation of memberRemark CLI command

chrlschwb commented 11 months ago

The user inputs member_id and is_verified, then the code encodes it to be used in the workerRemark extrinsic

This. For reference, please look at the implementation of memberRemark CLI command

I have changed as required

thesan commented 9 months ago

I'm closing this one in favour of #5057