Skip to content

feat: verify address according to chain type#1071

Merged
ashchan merged 4 commits intonervosnetwork:developfrom
Keith-CY:strengthen-address-validation
Nov 14, 2019
Merged

feat: verify address according to chain type#1071
ashchan merged 4 commits intonervosnetwork:developfrom
Keith-CY:strengthen-address-validation

Conversation

@Keith-CY
Copy link
Collaborator

No description provided.

@classicalliu
Copy link
Contributor

Conflicts and CI failed.

@Keith-CY Keith-CY removed the request for review from ashchan November 14, 2019 07:27
@Keith-CY Keith-CY changed the title feat: verify address according to chain type [HOLD} feat: verify address according to chain type Nov 14, 2019
nervos-bot[bot]
nervos-bot bot previously requested changes Nov 14, 2019
Copy link

@nervos-bot nervos-bot bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hold as requested by @Keith-CY.

@Keith-CY Keith-CY changed the title [HOLD} feat: verify address according to chain type [HOLD] feat: verify address according to chain type Nov 14, 2019
@Keith-CY Keith-CY changed the title [HOLD] feat: verify address according to chain type feat: verify address according to chain type Nov 14, 2019
@nervos-bot nervos-bot bot dismissed their stale review November 14, 2019 08:45

Unhold as requested by @Keith-CY.


const isAffordable = verifyTotalAmount(totalAmount, fee, balance)
const network = networks.find(n => n.id === networkID)
const isMainnet = (network && network.chain === 'ckb') || false // TODO: should be replaced after the mainnet tag is introduced
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use the mainnet tag feature which was recently merged.

@ashchan ashchan merged commit e17c47c into nervosnetwork:develop Nov 14, 2019
@Keith-CY Keith-CY deleted the strengthen-address-validation branch November 14, 2019 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants