<p>I think there are 2 types of review phases:</p>
<ol>
<li>The requester awaits feedback and is able to change the proposal. Best would be in cases of uncertain amounts and/or bigger projects to start an early discussion in which range the amount should be and bring different valuation aspects (like <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=5642225" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/joachimneumann">@joachimneumann</a> did in his last proposal).</li>
<li>After the proposal time is over, there should be still time so that other contributors can add comments, give feedback, discuss.... This will not help the proposal maker to change it as it is too late but it will help other voters to find a decision. There might be one or 2 contributors who have more insight into the topic of the request and here they can add information or their opinion. Ideally that happens already in the first review phase, so the contributor has time to adjust. Here if the discussion leads to much disagreement, the proposal might get rejected and need to be re-submitted next month.</li>
</ol>
<p>I consider the first review as the important one but here we have no technical means to enforce or clearly define it. The second one we could easily implement with the BREAK1 phase between the proposal phase and the blind vote phase. I think it is good to make that break a longer as it was initially planned for re-org protection (10 blocks). I would suggest 300 blocks here (about 2 day).</p>
<p>The first review phase we can support in a soft form:<br>
Lets keep the last 3 days of the proposal phase reserved for review. If the user makes a new proposal here he gets displayed a popup explaining that it is recommended to make proposals early and by submitting a "late proposal" he risks that reviewers will suggest to re-apply it for next months if they don't find enough time to review it. I think that does enough education to make sure that most contributors will follow the guideline to make the proposals before that "late" phase.</p>
<p>Please keep in mind that as the time will be based on blocks we have no way to sync with calendar months! i think it might be a good idea to keep blocks to even numbers like 150 block for 1 day instead of 144 blocks to make it easier to remember phase changes. The UI will show of course those events and also the estimated date based on the average 10 min. block interval. If we start the genesis block at an even number as well we might stick to a certain grid.</p>
<p>Periods can be changed by voting so that will not be a hard restriction anyway. I will make a new proposal to suggest the lengths of those periods but it will be roughly based on a 1 months cycle.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/bisq-network/proposals/issues/40#issuecomment-428628078">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZtoAc5V3R4E7NyULSFJIqRCyZxWHaks5ujhhqgaJpZM4W4xDT">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtsg-vbMob3WscJrxW5hviElzpQCLks5ujhhqgaJpZM4W4xDT.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/bisq-network/proposals","title":"bisq-network/proposals","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/bisq-network/proposals"}},"updates":{"snippets":[{"icon":"PERSON","message":"@ManfredKarrer in #40: I think there are 2 types of review phases:\r\n1. The requester awaits feedback and is able to change the proposal. Best would be in cases of uncertain amounts and/or bigger projects to start an early discussion in which range the amount should be and bring different valuation aspects (like @joachimneumann did in his last proposal).\r\n2. After the proposal time is over, there should be still time so that other contributors can add comments, give feedback, discuss.... This will not help the proposal maker to change it as it is too late but it will help other voters to find a decision. There might be one or 2 contributors who have more insight into the topic of the request and here they can add information or their opinion. Ideally that happens already in the first review phase, so the contributor has time to adjust. Here if the discussion leads to much disagreement, the proposal might get rejected and need to be re-submitted next month.\r\n\r\nI consider the first review as the important one but here we have no technical means to enforce or clearly define it. The second one we could easily implement with the BREAK1 phase between the proposal phase and the blind vote phase. I think it is good to make that break a longer as it was initially planned for re-org protection (10 blocks). I would suggest 300 blocks here (about 2 day).\r\n\r\nThe first review phase we can support in a soft form:\r\nLets keep the last 3 days of the proposal phase reserved for review. If the user makes a new proposal here he gets displayed a popup explaining that it is recommended to make proposals early and by submitting a \"late proposal\" he risks that reviewers will suggest to re-apply it for next months if they don't find enough time to review it. I think that does enough education to make sure that most contributors will follow the guideline to make the proposals before that \"late\" phase. \r\n\r\nPlease keep in mind that as the time will be based on blocks we have no way to sync with calendar months! i think it might be a good idea to keep blocks to even numbers like 150 block for 1 day instead of 144 blocks to make it easier to remember phase changes. The UI will show of course those events and also the estimated date based on the average 10 min. block interval. If we start the genesis block at an even number as well we might stick to a certain grid. \r\n\r\nPeriods can be changed by voting so that will not be a hard restriction anyway. I will make a new proposal to suggest the lengths of those periods but it will be roughly based on a 1 months cycle.  "}],"action":{"name":"View Issue","url":"https://github.com/bisq-network/proposals/issues/40#issuecomment-428628078"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/proposals/issues/40#issuecomment-428628078",
"url": "https://github.com/bisq-network/proposals/issues/40#issuecomment-428628078",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [bisq-network/proposals] Separate compensation request submission and voting periods (#40)",
"sections": [
{
"text": "",
"activityTitle": "**Manfred Karrer**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@ManfredKarrer",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"bisq-network/proposals\",\n\"issueId\": 40,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close issue",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueClose\",\n\"repositoryFullName\": \"bisq-network/proposals\",\n\"issueId\": 40\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/bisq-network/proposals/issues/40#issuecomment-428628078"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 383979731\n}"
}
],
"themeColor": "26292E"
}
]</script>