Skip to content
This repository has been archived by the owner on Jun 15, 2021. It is now read-only.

Add doc for compensation requests and voting #104

Merged
merged 2 commits into from
Feb 16, 2019
Merged

Add doc for compensation requests and voting #104

merged 2 commits into from
Feb 16, 2019

Conversation

m52go
Copy link
Contributor

@m52go m52go commented Feb 15, 2019

This doc is fairly short, but let me know if it would be better to include a more brief overview at the start.

Copy link
Contributor

@ManfredKarrer ManfredKarrer left a comment

Choose a reason for hiding this comment

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

utACK

====
Bisq contributions are valued _on completion_. The metric for that is _merged to master_ or otherwise _already delivered_ if the contribution is not code. How you determine the value you request is up to you, but it should be based on the value of the contribution you made, not on the raw time you spent.

For example, it might be reasonable to request 100 BSQ for fixing some typos in a doc. But requesting 1000 BSQ for that same task, just because it took you a few hours to read through it, will probably be rejected.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe add something like:
If you are unsure about the estimation for the value of your contribution ask youself what you would charge for that work if you would have done it for any company as freelancer.

Also fix small instance of awkward wording.
@m52go m52go mentioned this pull request Feb 16, 2019
@m52go
Copy link
Contributor Author

m52go commented Feb 16, 2019

Only material change I made was implement ManfredKarrer's suggestion, so I will merge this now.

This will allow #108 to be merged as soon as it's reviewed.

@m52go m52go merged commit 1456936 into bisq-network:master Feb 16, 2019
m52go pushed a commit that referenced this pull request Feb 16, 2019
@m52go m52go deleted the add-cr-details branch February 27, 2019 14:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants