<p></p>
<p>Good questions, thanks. I'd like the overall model and feel of bisq-cli to match that of bitcoin-cli, but not slavishly so. As we progress, we'll probably find many reasons to diverge in ways big and small. In this particular case, I find the wallet encryption-related bitcoind rpc method names and design to be somewhat haphazard, i.e. not particularly coherent or intuitive. It's good to be informed by what bitcoind has done, so as not to naively reinvent things, but having been so informed, I'm happy to take a fresh cut at our own equivalents while we have a relatively blank design slate. And note that we should deviate in any case from the 'passphrase' term they use to the 'password' phrase we have used throughout Bisq's history and UI.</p>
<blockquote>
<p>wouldn't it be nice if we stuck to bitcoin-cli's method and parameter names where appropriate? If we did, many new users would intuitively know how to use bisq-cli without even having to look at --help.</p>
</blockquote>
<p>Users will need to refer to --help output or other documentation frequently when using the Bisq rpc api and cli whether or not the methods in question closely match their bitcoind analogs. The overlap in these particular methods exists only because Bisq and Bitcoin Core share a common 'wallet' subdomain; as we get deeper into designing methods specific to Bisq's trading domain, our two APIs will differ greatly. More important than parity with bitcoind's api design is that our own api design is internally consistent, following a set of design principles that allow users to intuit and predict what functionality they need is likely to be called, and where that functionality does not yet exist, to afford contributors the ability to name and design things correctly out of the gate according to those principles and conventions.</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/4189#issuecomment-619567504">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNQQSTCIMUIPDW3T75DRORFM5ANCNFSM4MOQY5SQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNUFF7O7LEVMNLEZAWTRORFM5A5CNFSM4MOQY5S2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOETW5TEA.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/4189#issuecomment-619567504",
"url": "https://github.com/bisq-network/bisq/pull/4189#issuecomment-619567504",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>