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 select-pool-to-delegate.md #101

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ADARobinHood
Copy link

The original guide contained some personal biases. This proposed change prioritizes network health and delegator return on stake.

The original guide contained some personal biases. This proposed change prioritizes network health and delegator return on stake.
@rdlrt
Copy link
Contributor

rdlrt commented Oct 13, 2022

Thanks for the PR, in general - I like the content updates.

But we need to remember these are aimed at an end user in crypto ecosystem (a big % of which unfortunately seem allergic to read long pages), and would be nice if we use headings for sections - as they also become direct links that can be shared, for instance - for instance, when someone asks "what pool margin means"


* [Pooltool](https://pooltool.io)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dont think the explorer references should be removed, an end user wouldn't know where to look at and compare between pools

@ADARobinHood
Copy link
Author

Thanks for your review!

I agree, in current form this new guide lacks readability. Perhaps adding a quick reference section at the top that condenses the most relevant information to be readable within a minute or two would help.

There still needs to be a long wind version for people willing to take the time to go deeper. Formatting can still be improved in the long version.

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