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

<p>utACK. This looks fine as-is, but I've noticed that if my Bitcoin Core node is <em>not</em> fully synced, that Bisq falls back to connecting to Bisq's federation of nodes. This is basically fine from a privacy perspective (assuming the user trusts our nodes), but it always surprises me when it happens, and it would be nice (I think) if Bisq gave me some kind of heads up that it's going to fall back to the federation, with the choice of letting it go ahead or quitting Bisq while waiting for my Bitcoin node to finish syncing. Not saying that we need to hold up this PR at all; just wanted to put this out there, since I'd been thinking about it lately anyway, and because it's related to this change.</p>
<p>See also my review comment at <a class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="329617818" data-permission-text="Issue title is private" data-url="https://github.com/bisq-network/bisq-core/issues/118" href="https://github.com/bisq-network/bisq-core/pull/118#pullrequestreview-126301234">#118 (review)</a> regarding avoiding creating a separate issue and pull request pairs where a single pull request would suffice. Thanks.</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-core/pull/119#pullrequestreview-126302873">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZtjW9yeT8FiH_fWT79xASL4n_AfsWks5t56MGgaJpZM4UbkKz">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtiOOX8x9IJhglYth-3GVenl0TIHAks5t56MGgaJpZM4UbkKz.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-core/pull/119#pullrequestreview-126302873","url":"https://github.com/bisq-network/bisq-core/pull/119#pullrequestreview-126302873","name":"View Pull Request"},"description":"View this Pull Request on GitHub","publisher":{"@type":"Organization","name":"GitHub","url":"https://github.com"}}</script>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/bisq-network/bisq-core","title":"bisq-network/bisq-core","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/bisq-network/bisq-core"}},"updates":{"snippets":[{"icon":"PERSON","message":"@cbeams approved #119"}],"action":{"name":"View Pull Request","url":"https://github.com/bisq-network/bisq-core/pull/119#pullrequestreview-126302873"}}}</script>
<script type="application/ld+json">{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "37567f93-e2a7-4e2a-ad37-a9160fc62647",
"title": "@cbeams approved 119",
"sections": [
{
"text": "utACK. This looks fine as-is, but I've noticed that if my Bitcoin Core node is _not_ fully synced, that Bisq falls back to connecting to Bisq's federation of nodes. This is basically fine from a privacy perspective (assuming the user trusts our nodes), but it always surprises me when it happens, and it would be nice (I think) if Bisq gave me some kind of heads up that it's going to fall back to the federation, with the choice of letting it go ahead or quitting Bisq while waiting for my Bitcoin node to finish syncing. Not saying that we need to hold up this PR at all; just wanted to put this out there, since I'd been thinking about it lately anyway, and because it's related to this change.\r\n\r\nSee also my review comment at https://github.com/bisq-network/bisq-core/pull/118#pullrequestreview-126301234 regarding avoiding creating a separate issue and pull request pairs where a single pull request would suffice. Thanks.",
"activityTitle": "**Chris Beams**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@cbeams",
"facts": [

]
}
],
"potentialAction": [
{
"targets": [
{
"os": "default",
"uri": "https://github.com/bisq-network/bisq-core/pull/119#pullrequestreview-126302873"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 342770355\n}"
}
],
"themeColor": "26292E"
}</script>