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

<p>NACK<br>
I don't see the need for excluding the other fields. It add some unneeded risk and testing effort. The objects should be immutable so any fields has to be fix. If we go further on that area I would recommend to think how we can refactor to make all those accounts immutable. But I think there are many other higher prio stuff to do....</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/3572?email_source=notifications&email_token=AJFFTNRFV3L5H2W6JXSKA3DQT4H4DA5CNFSM4JKDCHK2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCLZHAMI#pullrequestreview-317878321">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNUAD2PQN6S37YT5W7LQT4H4DANCNFSM4JKDCHKQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNR5OCKCYMHTBSHJK3DQT4H4DA5CNFSM4JKDCHK2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCLZHAMI.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/3572?email_source=notifications\u0026email_token=AJFFTNRFV3L5H2W6JXSKA3DQT4H4DA5CNFSM4JKDCHK2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCLZHAMI#pullrequestreview-317878321",
"url": "https://github.com/bisq-network/bisq/pull/3572?email_source=notifications\u0026email_token=AJFFTNRFV3L5H2W6JXSKA3DQT4H4DA5CNFSM4JKDCHK2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCLZHAMI#pullrequestreview-317878321",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>