Upgrade SDKs and Librustzcash, for both Android and iOS #220
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…act-native metho in WalletFolder.js mocks
- Between verusmobile and react-native-verus
- Need to clean a lot of this up. Many different useless bits and pieces in here
…into maven-verus
- with verus-android-wallet-sdk artifacts
who-biz
commented
Oct 3, 2025
| } | ||
| } | ||
| signingConfigs { | ||
| /*signingConfigs { |
Contributor
Author
There was a problem hiding this comment.
This should be re-enabled for prod release.
who-biz
commented
Oct 3, 2025
| @@ -1,5 +1,5 @@ | |||
| let dlightServers = { | |||
| vrsc: ['lightwallet.verus.services:8120'], | |||
| vrsc: ['lightwalletd.whobiz.dev:443'], | |||
Contributor
Author
There was a problem hiding this comment.
This also should be changed back before prod.
who-biz
force-pushed
the
dev-backend-upgrade
branch
from
7849b59 to
c080f51
Compare
who-biz
commented
Oct 3, 2025
| pubKey: null, | ||
| privKey: spendingKey, | ||
| viewingKey: await VerusLightClient.deriveViewingKey(spendingKey), | ||
| viewingKey: viewKey, |
Contributor
Author
There was a problem hiding this comment.
Should discuss if this one needs changed.
I don't know that we use V0 anywhere, but our encoded viewkey is a unified key, now (uview.... bech32m-encoded).
Easiest solution is just to not store the viewkey, since we compute these internally in RN functions that need it, from spendkey or seed.
who-biz
force-pushed
the
dev-backend-upgrade
branch
from
55e4335 to
8734a61
Compare
who-biz
force-pushed
the
dev-backend-upgrade
branch
from
c0eb0ca to
708e7f3
Compare
who-biz
force-pushed
the
dev-backend-upgrade
branch
from
c765787 to
e6c05d5
Compare
- Also throw on hex import, since we don't yet support it on iOS - Tools.bech32Decode will also now properly propagate its if we fail to decode the string
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR includes the supporting code for migration of Verus-mobile to:
1.) new
react-native-verusdependency (replacesverus-light-client)2.) newer versions of the SDKs for each platform
3.) newer
librustzcashas wellThis PR does not include the encryption support work done by Artist.
No known issues/errors at time of writing, but still testing.