<blockquote>
<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=20237127" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/niyid">@niyid</a></p>
<p>Wouldn't you end up essentially re-writing MoneroWalletRpc.java? The only thing extra that class does is translates monero-wallet-rpc's hodgepodge API to a clear model definition intended for long term use. The purpose of the library is to avoid the need for application developers to roll their own binding with non-uniform APIs. The library already provides extensive tests, a clear model definition, multisignature support, and the ability to switch to a native wallet binding with the same API. I do not see why we would not use that as a matter of code review.</p>
<p>I am working on uploading the <code>monero-java</code> project to the Maven repository. I can change the POM to package the project as POM instead of JAR, if that would be useful.</p>
<p>FYI we're on Bisq's Slack workspace.</p>
</blockquote>
<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=13068859" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/woodser">@woodser</a></p>
<p>As I already pointed out, I am quite with the inclusion of <strong><code>monero-java</code></strong> as a library even if with respect to the current goal (which is to have a bridge to <strong><code>monero-wallet-rpc</code></strong>. I was also of the naive opinion that it will be accepted without a code review based on your role on Bisq. The <strong><code>monero-java</code></strong> is no doubt a robust library that provides the current needs with respect to the goal of the current PR and proposal. While many of the extensive functionality of <strong><code>monero-java</code></strong> may not be in use at the moment, they may prove very useful in the near and far future.</p>
<p>But with the requirement of a code review being placed on <strong><code>monero-java</code></strong>, this PR is dead in the water as I do not see how 2 or 3 people can review a project of tens of thousands of files and functions within a reasonable time.</p>
<p>On the matter of rewriting <code>MoneroWalletRpc.java</code>, not at all. It is too robust for what I have in mind as it even references <code>MoneroWalletDaemon</code>.</p>
<p>I will join you on Bisq Slack soon.</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/3275?email_source=notifications&email_token=AJFFTNUS5KD7ULMWNTQIA4TQKJAXJA5CNFSM4IXG65PKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7ANNFI#issuecomment-532731541">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNQC35MESQZ7UNAJVZ3QKJAXJANCNFSM4IXG65PA">mute the thread</a>.<img src="https://github.com/notifications/beacon/AJFFTNQDIP54BGVPAUHJSSLQKJAXJA5CNFSM4IXG65PKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7ANNFI.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/3275?email_source=notifications\u0026email_token=AJFFTNUS5KD7ULMWNTQIA4TQKJAXJA5CNFSM4IXG65PKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7ANNFI#issuecomment-532731541",
"url": "https://github.com/bisq-network/bisq/pull/3275?email_source=notifications\u0026email_token=AJFFTNUS5KD7ULMWNTQIA4TQKJAXJA5CNFSM4IXG65PKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7ANNFI#issuecomment-532731541",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>