<p>It's really about being pragmatic here. This implementation as-is adds 2K+ lines of code to a codebase that currently defines hundreds of assets while remaining under 10K LoC total. The next longest class in the codebase is 10x shorter than the one in this implementation. It just doesn't work, in the name of full validation to grow the codebase by 20% for a single new asset.</p>
<p>A regex that captures the allowed prefixes and underscore, validates that everything is lowercase, and that the address is of the correct length will go a long way toward doing 'enough' validation. If we find that Nano is a huge hit on Bisq, and we start to see validation errors cropping up due to invalid checksums, we can revisit a more complete implementation then.</p>
<p>Note that we're going to ship v0.7.1 within the next couple days, so if you can touch this up in the meantime, we can still get it in by the release. Best regards.</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-assets/pull/32#issuecomment-400661158">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZtuSoDvShh-1u-WDnREaZufcocL8gks5uA4BfgaJpZM4UhMe8">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtrVWg7InWAKqJS97G-fNngLGRkytks5uA4BfgaJpZM4UhMe8.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/bisq-assets","title":"bisq-network/bisq-assets","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-assets"}},"updates":{"snippets":[{"icon":"PERSON","message":"@cbeams in #32: It's really about being pragmatic here. This implementation as-is adds 2K+ lines of code to a codebase that currently defines hundreds of assets while remaining under 10K LoC total. The next longest class in the codebase is 10x shorter than the one in this implementation. It just doesn't work, in the name of full validation to grow the codebase by 20% for a single new asset.\r\n\r\nA regex that captures the allowed prefixes and underscore, validates that everything is lowercase, and that the address is of the correct length will go a long way toward doing 'enough' validation. If we find that Nano is a huge hit on Bisq, and we start to see validation errors cropping up due to invalid checksums, we can revisit a more complete implementation then.\r\n\r\nNote that we're going to ship v0.7.1 within the next couple days, so if you can touch this up in the meantime, we can still get it in by the release. Best regards."}],"action":{"name":"View Pull Request","url":"https://github.com/bisq-network/bisq-assets/pull/32#issuecomment-400661158"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq-assets/pull/32#issuecomment-400661158",
"url": "https://github.com/bisq-network/bisq-assets/pull/32#issuecomment-400661158",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [bisq-network/bisq-assets] List Nano (NANO) (#32)",
"sections": [
{
"text": "",
"activityTitle": "**Chris Beams**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@cbeams",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"bisq-network/bisq-assets\",\n\"issueId\": 32,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close pull request",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"PullRequestClose\",\n\"repositoryFullName\": \"bisq-network/bisq-assets\",\n\"pullRequestId\": 32\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/bisq-network/bisq-assets/pull/32#issuecomment-400661158"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 344246204\n}"
}
],
"themeColor": "26292E"
}
]</script>