<p><b>@freimair</b> approved this pull request.</p>

<p>utAck</p>
<p>I am not sure if we should keep adding code and complexity to the project like this. Sure, there is an issue there that needs fixing, hence, utack. However, it seems to me that there is a simpler solution just around the corner.</p><hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/3155#discussion_r318542827">core/src/main/java/bisq/core/dao/governance/proposal/ProposalListPresentation.java</a>:</p>
<pre style='color:#555'>> +
+
+    ///////////////////////////////////////////////////////////////////////////////////////////
+    // DaoSetupService
+    ///////////////////////////////////////////////////////////////////////////////////////////
+
+    @Override
+    public void addListeners() {
+    }
+
+    @Override
+    public void start() {
+        // We must set the listeners initially and not on onParseBlockChainComplete as activeOrMyUnconfirmedProposals
+        // is used in voteResults which can be called earlier during sync.
+        // To avoid unneeded upDateLists calls we delay one render frame so that once the proposalService is complete we
+        // register out listeners.
</pre>
⬇️ Suggested change
<pre style="color: #555">-        // register out listeners.
+        // register our listeners.
</pre>


<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/3155#discussion_r318543555">core/src/main/java/bisq/core/dao/governance/proposal/ProposalListPresentation.java</a>:</p>
<pre style='color:#555'>> +    }
+
+
+    ///////////////////////////////////////////////////////////////////////////////////////////
+    // DaoSetupService
+    ///////////////////////////////////////////////////////////////////////////////////////////
+
+    @Override
+    public void addListeners() {
+    }
+
+    @Override
+    public void start() {
+        // We must set the listeners initially and not on onParseBlockChainComplete as activeOrMyUnconfirmedProposals
+        // is used in voteResults which can be called earlier during sync.
+        // To avoid unneeded upDateLists calls we delay one render frame so that once the proposalService is complete we
</pre>
⬇️ Suggested change
<pre style="color: #555">-        // To avoid unneeded upDateLists calls we delay one render frame so that once the proposalService is complete we
+        // To avoid unneeded updateLists(); calls, we delay one render frame so that once the proposalService is complete, we
</pre>


<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/3155#discussion_r318544736">common/src/main/java/bisq/common/app/Capability.java</a>:</p>
<pre style='color:#555'>> @@ -31,7 +31,7 @@
     PROPOSAL,
     BLIND_VOTE,
     ACK_MSG,
-    BSQ_BLOCK,
+    RECEIVE_BSQ_BLOCK,
</pre>
<p>I am not sure we should change stuff like that. What I am sure of is that we should add Javadoc to describe what a capability is for and what it does, only so can we make sure devs can understand what these 2 or 3 words mean.</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/bisq/pull/3155?email_source=notifications&email_token=AJFFTNTZLNPM77OL6RK3UCTQGZT73A5CNFSM4IQ5ZKIKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCC6F7FQ#pullrequestreview-280780694">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNVCHE5CLGJILUZ5WNTQGZT73ANCNFSM4IQ5ZKIA">mute the thread</a>.<img src="https://github.com/notifications/beacon/AJFFTNVZN5ZQMYLBVZMXQOLQGZT73A5CNFSM4IQ5ZKIKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCC6F7FQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq/pull/3155?email_source=notifications\u0026email_token=AJFFTNTZLNPM77OL6RK3UCTQGZT73A5CNFSM4IQ5ZKIKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCC6F7FQ#pullrequestreview-280780694",
"url": "https://github.com/bisq-network/bisq/pull/3155?email_source=notifications\u0026email_token=AJFFTNTZLNPM77OL6RK3UCTQGZT73A5CNFSM4IQ5ZKIKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCC6F7FQ#pullrequestreview-280780694",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>