Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

url.ts now references JSON files instead of static URL array. #609

Closed
wants to merge 22 commits into from
Closed

url.ts now references JSON files instead of static URL array. #609

wants to merge 22 commits into from

Conversation

kjlevitz
Copy link

url.ts now references JSON files instead of static URL array.

Reference #596 (getNodes)

Currently working through multi-sig, there are a few comments noting what needs to be tweaked.
Currently working through multi-sig, there are a few comments noting what needs to be tweaked.
Change size calculator for size() function.
Removed Callback logic
Addressed changes suggested from previous commit. 
Currently tested with:
Single Sig: 0C21aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa0B4195440d78
Multi Sig: 0102000c21dddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddd0c21dddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddd0c21dddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddddd0103000b41138defaf

Both pass
Resolve Linting Error
Linting fix
camelCase for function names and variables
camelCase for functions and variables, rename functions for signatureVerification.ts calls.
@kjlevitz kjlevitz closed this Oct 19, 2020
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.

1 participant