<p>For me ideal option would be if I had permission to edit those PRs prepared by external devs. That would make the process fastest.<br>
On the other hand, I think that we cannot trust that those validators delivered by 3rd parties are correct and can potentially reject addresses that are correct.<br>
I think that safest way is to check if address is not empty string and that's all.</p>
<p>So I'm for dropping requirement of providing address validator in asset listing PR.</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/proposals/issues/67#issuecomment-454695282">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZtr0zTFkPwSuAmBLNG5r4Q43rjyB4ks5vDuRrgaJpZM4aBSOV">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtnluh-h18nb1lwsYVRWi_WLVrGlPks5vDuRrgaJpZM4aBSOV.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/bisq-network/proposals","title":"bisq-network/proposals","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/bisq-network/proposals"}},"updates":{"snippets":[{"icon":"PERSON","message":"@blabno in #67: For me ideal option would be if I had permission to edit those PRs prepared by external devs. That would make the process fastest.\r\nOn the other hand, I think that we cannot trust that those validators delivered by 3rd parties are correct and can potentially reject addresses that are correct.\r\nI think that safest way is to check if address is not empty string and that's all.\r\n\r\nSo I'm for dropping requirement of providing address validator in asset listing PR."}],"action":{"name":"View Issue","url":"https://github.com/bisq-network/proposals/issues/67#issuecomment-454695282"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/proposals/issues/67#issuecomment-454695282",
"url": "https://github.com/bisq-network/proposals/issues/67#issuecomment-454695282",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>