<p><b>@chimp1984</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/3212#discussion_r321653258">desktop/src/main/java/bisq/desktop/util/GUIUtil.java</a>:</p>
<pre style='color:#555'>> @@ -1022,4 +1050,22 @@ public static void openTxInBsqBlockExplorer(String txId, Preferences preferences
         if (txId != null)
             GUIUtil.openWebPage(preferences.getBsqBlockChainExplorer().txUrl + txId, false);
     }
+
+    public static String getBsqInUsd(Price bsqPrice,
</pre>
<p>You mean getBSQInUsd instead of getBsqInUsd? In the DAO where it appears often we dont use uppercase. In general we dont use all uppercase (getApiKey not getAPIKey) for  abbreviation as the following Camelcase letter might make the abbreviation harder to read. Of course question of tase and opinon, but code base is follwing that style largely.</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/3212?email_source=notifications&email_token=AJFFTNUXOA7BL2CSOBOEOGLQIIPMFA5CNFSM4IUELKQKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCD4QKEA#discussion_r321653258">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNXZYJAL33ZVCT26GXLQIIPMFANCNFSM4IUELKQA">mute the thread</a>.<img src="https://github.com/notifications/beacon/AJFFTNTMSQIABDMWPW2D7FLQIIPMFA5CNFSM4IUELKQKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCD4QKEA.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/3212?email_source=notifications\u0026email_token=AJFFTNUXOA7BL2CSOBOEOGLQIIPMFA5CNFSM4IUELKQKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCD4QKEA#discussion_r321653258",
"url": "https://github.com/bisq-network/bisq/pull/3212?email_source=notifications\u0026email_token=AJFFTNUXOA7BL2CSOBOEOGLQIIPMFA5CNFSM4IUELKQKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCD4QKEA#discussion_r321653258",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>