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

pricing paper on front pages & small faq modification #2520

Merged
merged 3 commits into from
Jan 15, 2021

Conversation

arjunhassard
Copy link
Member

@arjunhassard arjunhassard commented Jan 14, 2021

Type of PR:

  • Bugfix
  • Feature
  • Documentation
  • Other

Required reviews:

  • 1
  • 2
  • 3

What this does:
Makes resource on network fees more accessible

Issues fixed/closed:
None

Why it's needed:
This doesn't address any productdev issues!

Notes for reviewers:
None, should be easy to approve!

Copy link
Member

@derekpierre derekpierre left a comment

Choose a reason for hiding this comment

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

Looks good - but needs a newsfragment - see https://github.com/nucypher/nucypher/blob/main/newsfragments/README.md.

TL;DR create a file in nucypher/newsfragments named 2520.doc.rst with a one-liner outlining the crux of the change.

@@ -140,7 +140,7 @@ Inflation Rewards (NU) and Policy Fees (ETH).
Q: How are Policy Fees (ETH) determined?
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

The fee is calculated based on each period committed to and the number of policies that the Ursula is enforcing.
The amount of fees earnable by a staker in a given period is determined by multiple factors, including (1) the minimum fee rate set by the staker, (2) the number of active policies issued by users of the network to that staker, and (3) whether the staker has previously committed to being available for that period. See `Service Fees (Pricing) <https://docs.nucypher.com/en/latest/architecture/service_fees.html>`_ for details.
Copy link
Member

Choose a reason for hiding this comment

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

@arjunhassard I'm never one to question your vocabulary prowess 😅 - but is "earnable" a word?

Copy link
Member

Choose a reason for hiding this comment

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

"Earnable" sounds good to me, and seems to have at least some marginal use for those that speak legalese, but I guess we can phrase it in a simpler way:

Suggested change
The amount of fees earnable by a staker in a given period is determined by multiple factors, including (1) the minimum fee rate set by the staker, (2) the number of active policies issued by users of the network to that staker, and (3) whether the staker has previously committed to being available for that period. See `Service Fees (Pricing) <https://docs.nucypher.com/en/latest/architecture/service_fees.html>`_ for details.
The amount of fees a staker can earn in a given period is determined by multiple factors, including (1) the minimum fee rate set by the staker, (2) the number of active policies issued by users of the network to that staker, and (3) whether the staker has previously committed to being available for that period. See `Service Fees (Pricing) <https://docs.nucypher.com/en/latest/architecture/service_fees.html>`_ for details.

Copy link
Member

Choose a reason for hiding this comment

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

The FAQ has turned into a nursery for actual documentation.

Copy link
Member Author

Choose a reason for hiding this comment

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

@KPrasch Isn't that kind of what an FAQ is for? To give users a quick, simplified answer without having to trudge through the whole spec – thereby addressing high-level confusion before a rage/exhaustion-quit?

Do you think this change makes it more or less of a nursery? I corrected a misleading answer and added a link to the actual documentation.

Copy link
Member

Choose a reason for hiding this comment

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

I'm not saying it's a bad thing, just an observation. Even before this PR, the FAQ encompasses too much essential information to be properly composed and discovered here. I even think it's part of the lifecycle of documentation for our organization, certainly better to have some kind of reference in the form of an FAQ than none at all.

Copy link
Member Author

Choose a reason for hiding this comment

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

@KPrasch agreed, it's particularly intimidating to a fresh developer/user to have to navigate through so much information primarily aimed at stakers. been saying for a long time we need to split all our educational materials by audience (demand-side and service-side) – maybe now's the time?

Copy link
Member

Choose a reason for hiding this comment

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

Our old friend #1001

Copy link
Member

Choose a reason for hiding this comment

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

Some initial work by @KPrasch , #2463 .

arjunhassard and others added 2 commits January 14, 2021 12:15
Co-authored-by: David Núñez <david@nucypher.com>
@arjunhassard
Copy link
Member Author

@derekpierre

Looks good - but needs a newsfragment - see https://github.com/nucypher/nucypher/blob/main/newsfragments/README.md.

TL;DR create a file in nucypher/newsfragments named 2520.doc.rst with a one-liner outlining the crux of the change.

Done, thanks for heads-up!

Copy link
Member

@derekpierre derekpierre left a comment

Choose a reason for hiding this comment

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

🎸

@KPrasch KPrasch merged commit 922b03e into nucypher:main Jan 15, 2021
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.

4 participants