Skip to content
This repository has been archived by the owner on Sep 14, 2023. It is now read-only.

chore: crypto util / wasm binding cleanup #116

Merged
merged 7 commits into from
Jun 17, 2022
Merged

chore: crypto util / wasm binding cleanup #116

merged 7 commits into from
Jun 17, 2022

Conversation

harrysolovay
Copy link
Contributor

@harrysolovay harrysolovay commented Jun 17, 2022 •
edited
Loading

  • Begin doing some chain-specific codegen for a soon-to-be-developed ss58 conversion util
  • Clean up how we expose/load wasm bindings
  • Clean up the sr25519 utils (most of which are solely for testing)
  • Bind to ss58 encoding/decoding implementation
  • Simplify usage of these bindings in transfer example
  • Fix type errors elsewhere
  • ErrorCtor util

@harrysolovay harrysolovay marked this pull request as ready for review June 17, 2022 15:53
karl-kallavus
karl-kallavus approved these changes Jun 17, 2022
View reviewed changes
Copy link
Contributor

@karl-kallavus karl-kallavus left a comment

Choose a reason for hiding this comment

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

🍰

@harrysolovay harrysolovay changed the title Crypto Util / Wasm Binding Cleanup chore: crypto util / wasm binding cleanup Jun 17, 2022
@harrysolovay harrysolovay merged commit f8b3ffa into main Jun 17, 2022
@harrysolovay harrysolovay deleted the ss58-util branch June 17, 2022 17:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@karl-kallavus karl-kallavus karl-kallavus approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@harrysolovay @karl-kallavus