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

Update registration tutorial #6

Closed
wants to merge 4 commits into from

Conversation

sacha-l
Copy link

@sacha-l sacha-l commented Jul 30, 2022

Suggestions to update the tutorial for registering a community on Gesell testnet.

For some reason I got an error when trying to complete the final steps:

 gesell new-community specfile.json 5D4fwokdcKUePqNSGvs9DBkbRycYc6VoDN7spjFv6L6u1HAz
error: error: Found argument '5D4fwokdcKUePqNSGvs9DBkbRycYc6VoDN7spjFv6L6u1HAz' which wasn't expected, or isn't valid in this context

I also tried doing this through Polkadot JS Apps UI and got a BadOrigin error. Haven't gone too far into trying to understand what's causing this but I suspect it's something in how the runtime expects the specfile.json or perhaps the account needs some minimal balance that the faucet is not providing?

Couple of things I'd like to put in before this gets merged:

  • What's next: let's point users to what they should look into after registration.
  • Link or explanation as to what the CID is / how to use it

Other nit picks that could make the experience much better in my opinion:

  • The CLI commands for what was nctr list-communities is very messy, what's a trick to display just what we want from the RPC return?
  • The new account command is not so user friendly because it just creates artifacts of the seed phrases inside the node project. Can you modify the CLI tool to also output the corresponding address in the keystore directory?

Hope this write-up helps other users!

@brenzi
Copy link
Member

brenzi commented Jul 31, 2022

Thanks for these hints! Indeed, our tutorial is outdated since we had to do a few mods for mainnet.

Currently, only the association can register new communities. We can fix that for Testnet Gesell with: encointer/pallets#251

@brenzi
Copy link
Member

brenzi commented Jul 31, 2022

regarding list-communities. I don't understand what's messy about it. can you explain @sacha-l ?

./encointer-client-notee -u wss://gesell.encointer.org -p 443 list-communities
number of communities:  5
e5dvt5mjcem: bot-tugs locations: 9
u0qj94fxxJ6: EdisonPaula locations: 3
srcq45PYNyD: Adriana locations: 5
u33e0719fDB: Decoded Berlin locations: 3
69y7j4ZEXmy: Decoded Buenos Aires locations: 8

The tutorial actually features an example with the old hash-based cid, which now looks more concise. We should change thjat in the tutorial

@brenzi
Copy link
Member

brenzi commented Jul 31, 2022

regarding new-account. thats most likely a won't fix for us and should be solved upstream. But they idea was lurking around my head since 2019 as I just realize ;-)
scs/substrate-api-client#266

@sacha-l
Copy link
Author

sacha-l commented Jul 31, 2022

Oh that's weird, I no longer get the same result (before I was getting a whole bunch of state query results from the json RPC). I can confirm that it displays nicely now:

> gesell list-communities
number of communities:  5
e5dvt5mjcem: bot-tugs locations: 9
u0qj94fxxJ6: EdisonPaula locations: 3
srcq45PYNyD: Adriana locations: 5
u33e0719fDB: Decoded Berlin locations: 3
69y7j4ZEXmy: Decoded Buenos Aires locations: 8

@sacha-l
Copy link
Author

sacha-l commented Aug 1, 2022

What's next: let's point users to what they should look into after registration.
Link or explanation as to what the CID is / how to use it

On these two TODOs could you point me to information about what you'd add?

@brenzi
Copy link
Member

brenzi commented Aug 1, 2022

WIP: #7

@brenzi
Copy link
Member

brenzi commented Aug 1, 2022

I have written up some up to date content in #7. Please rebase and check if you can use it or if you have further feedback

@sacha-l
Copy link
Author

sacha-l commented Aug 4, 2022

Looks like we can close this and I'll follow up with a review of your updates. Cheers!

@brenzi brenzi closed this Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants