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.

Sign up for GitHub

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

Jump to bottom

SDK: Add handling checksum for addresses #42

Open
ethedev opened this issue Aug 24, 2021 · 0 comments
Open

SDK: Add handling checksum for addresses #42

ethedev opened this issue Aug 24, 2021 · 0 comments
Assignees
@snake-poison
Labels
important Important
Milestone
New Synths

Comments

@ethedev
Copy link
Member

ethedev commented Aug 24, 2021 •
edited
Loading

After reading from asset json and generally everywhere, convert addresses to what is needed according to the function that is using it, for subgraph lowercasing it is needed for example you can lower case the checksummed address before passing it to subgraph.
@ethedev ethedev added the important Important label Aug 24, 2021
@ethedev ethedev changed the title SDK: Add checksum addresses after reading from asset json and generally everywhere SDK: Add handling checksum for addresses Aug 26, 2021
@ethedev ethedev added this to the New Synths milestone Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@snake-poison snake-poison

Labels
important Important
Projects
None yet
Milestone
New Synths
Development

No branches or pull requests
2 participants
@snake-poison @ethedev