Skip to content

Conversation

@justinkook
Copy link
Contributor

  • Can validate ethereum Uris

URIs based on EIP - 681 / EIP - 831

@justinkook justinkook force-pushed the feature/ethUri branch 8 times, most recently from 6109e0e to d737be0 Compare September 26, 2019 14:55
@justinkook justinkook changed the title Feature/eth uri WIP: Feature/eth uri Sep 26, 2019
MIKE1203
MIKE1203 previously approved these changes Sep 30, 2019
@justinkook justinkook force-pushed the feature/ethUri branch 2 times, most recently from b7b360e to 059f7e0 Compare October 1, 2019 14:41
@justinkook justinkook changed the title WIP: Feature/eth uri Feature/eth uri Oct 1, 2019
Copy link
Contributor

@matiu matiu left a comment

Choose a reason for hiding this comment

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

Please add tests.

It woud be nice to pull the test vectors from other implementation.

@justinkook justinkook force-pushed the feature/ethUri branch 2 times, most recently from 28ff2d4 to 029548b Compare October 1, 2019 17:03
micahriggan
micahriggan previously approved these changes Oct 1, 2019
Copy link
Contributor

@micahriggan micahriggan left a comment

Choose a reason for hiding this comment

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

LGTM

micahriggan
micahriggan previously approved these changes Oct 1, 2019
@justinkook justinkook force-pushed the feature/ethUri branch 3 times, most recently from 0ccedff to 6403820 Compare October 1, 2019 19:25
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.

4 participants