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

[TA-3386]: NewWalletFromMnemonic #30

Open
wants to merge 9 commits into
base: v1.x.x
Choose a base branch
from

Conversation

GuillemGarciaDev
Copy link

[TA-3386]: NewWalletFromMnemonic

This PR aims to add a new wallet method NewWalletFromMnemonic method.

Changes

address-codec

  • Fixed secp256k1 private key prefix

keypairs

  • Add secp256k1 crypto algorithm

xrpl

  • Add new method NewWalletFromMnemonic

Copy link

@GuillemGarciaDev GuillemGarciaDev added pkg:xrpl xrpl package pkg:address-codec address-codec package pkg:keypairs keypairs package labels Oct 3, 2024
Base automatically changed from refactor/packages/setup to v1.x.x October 3, 2024 15:47
Copy link

@florent-uzio florent-uzio left a comment

Choose a reason for hiding this comment

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

Looks great to me! Nice job!

@@ -3,7 +3,7 @@ on:
push:
branches: [ main ]
pull_request:
branches: [ main, 'v*' ]
branches: [ '**' ]

Choose a reason for hiding this comment

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

What does ** mean?


return &Wallet{
PublicKey: pubKey,
PrivateKey: fmt.Sprintf("00%s", privKey),

Choose a reason for hiding this comment

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

So this generation from a mnemonic only generates a secp256k1 key ? Is it possible to generate an ed25519 also ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg:address-codec address-codec package pkg:keypairs keypairs package pkg:xrpl xrpl package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants