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

<p>ACK</p>
<p>I tested just that use case and it fixes the problem.</p>
<p>The underlying problem is that the order how listeners are registered and the domains are initialized is not sufficiently clearly organized.<br>
SupportManager registers its listener in the constructor, others in onAllServicesInitialized.</p>
<p>Listener are moslty held in HashSets, so the order of execution is not related to the order how they get added, which is semantically correct as listeners should not rely on order of execution, but as we see the domains would require more precise order.</p>
<p>It would require a larger refactoring to get those processes better defined.<br>
I think the suggested fix carries probably low risk but all main use cases specailly those involving mailbox messages should be tested well.</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/5219#pullrequestreview-596412887">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNTYNJHGDJAXVHPDQZ3TAO5QDANCNFSM4YBSWI7Q">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNUHW5X736LDM4PBFBLTAO5QDA5CNFSM4YBSWI72YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOEOGITVY.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/5219#pullrequestreview-596412887",
"url": "https://github.com/bisq-network/bisq/pull/5219#pullrequestreview-596412887",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>