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

Bring src of sdk-core-wallet with history #490

Merged
merged 140 commits into from
Oct 2, 2024
Merged

Conversation

danielailie
Copy link
Contributor

@danielailie danielailie commented Oct 2, 2024

See #483.

Steps involved:

git clone [email protected]:multiversx/mx-sdk-js-core.git
git clone [email protected]:multiversx/mx-sdk-js-wallet.git

cd mx-sdk-js-wallet
git filter-repo --subdirectory-filter src --to-subdirectory-filter src-wallet --tag-rename 'src':'src-wallet'

cd ../mx-sdk-js-core

git checkout feat/unify
git checkout -b merge-wallet
git remote add wallet ../mx-sdk-js-wallet
git fetch wallet
git merge --allow-unrelated-histories wallet/main
git remote remove wallet
git push origin

How to review:

git clone -b merge-wallet [email protected]:multiversx/mx-sdk-js-core.git
git clone [email protected]:multiversx/mx-sdk-js-wallet.git
git diff --no-index mx-sdk-js-core/src-wallet mx-sdk-js-wallet/src

References:
https://stackoverflow.com/a/62096626/1475331

andreibancioiu and others added 24 commits May 24, 2023 13:43
Workaround: cast (convert) buffers to Uint8Array before calling noble functions
…-key

Added extra check when decrypting secret key
Remove some dev-dependencies (axios, in particular)
Accept "Uint8Array", in addition to "Buffer" (on the main flows: sign, verify)
Add "mnemonicToEntropy" and "entropyToMnemonic"
@danielailie danielailie self-assigned this Oct 2, 2024
@andreibancioiu andreibancioiu changed the title Bring src of sdk-core-wallet with histori Bring src of sdk-core-wallet with history Oct 2, 2024
@danielailie danielailie merged commit de45867 into feat/unify Oct 2, 2024
3 of 4 checks passed
@danielailie danielailie deleted the merge-wallet branch October 2, 2024 13:28
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.

9 participants