-
Notifications
You must be signed in to change notification settings - Fork 133
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 02-robots.md #221
Update 02-robots.md #221
Conversation
Please feel free to modify text and/or formatting as needed. Apologies in advance for any non-standard contribution practices!
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.
Hey @okada-blue,
Thanks a lot for giving a push to the Documentation work. Congratulations on your first PR ever! The text reads really well.
I have left a comment for a small change (just a line deletion). Will also serve as practice for more complex reviews :)
Let's make this the first ever non-dev rewarded PR. Please submit an invoice for 20K Sats. It's little, just a prove of appreciation, but one more step towards to working for Bitcoin :D
Woops, I provided the invoice on the commit comment. I've copied here as well: lnbc200u1p3s5v32pp5p62jxach58d48u6yg4tt7jms9pvs7m07j4m7dzk2stud6pa62u8qdqu2askcmr9wssx7e3q2dshgmmndp5scqzpgxqyz5vqsp5r7ta253wcwp8usfjdqf5tja474wx877lu49puvqhn5gktud50hvs9qyyssq3cygts9l8r2rhzrv89a6d4wz8ect8296fsp60y32y4k00ghav5fkqm8dvlgme49mjvvcn0a3s8kuw7mzxs4jkltjtn0g4ufxm28c5xqp2txpxu |
Under the hood, token creation in the RoboSats homepage is the process of generating and encrypting your PGP private key with your token on the client side of the app. The end-user requests from the RoboSats node an avatar and nickname generated from your encrypted token and returns to you the corresponding robot identity. See the graphic below: | ||
|
||
![RoboSats Identity Generation Pipeine](https://learn.robosats.com/assets/images/private/usergen-pipeline.png) |
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.
Misspelled "Pipeline" in the brackets. Missing an "L" and should read:
![RoboSats Identity Generation Pipeline](https://learn.robosats.com/assets/images/private/usergen-pipeline.png)
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.
Do you want to push a new commit to your branch fixing this typo and remove the "Include WIP" line? Then I will merge :)
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.
I think I pushed a new commit correctly.. let me know if I've not though. Sorry, full noob on display
23e5c1d22cc0521cee93fb0fed949dfd195195f55623a96558d63e19972009cb |
Update "Robots" section of learn.robosats.com
Please feel free to modify text and/or formatting as needed. Apologies in advance for any non-standard contribution practices!