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

chore(wallet): Upgrade react-native-keychain #1611

Merged
merged 2 commits into from
Mar 19, 2024

Conversation

coreyphillips
Copy link
Collaborator

@coreyphillips coreyphillips commented Mar 18, 2024

Description

  • Upgrades to the refactor/android-data-store branch of react-native-keychain.

Type of change

  • Dependency Upgrade

Tests

  • No test

Upgrades to the refactor/android-data-store branch of react-native-ldk.
@coreyphillips coreyphillips added the enhancement New feature or request label Mar 18, 2024
@coreyphillips coreyphillips added this to the Audit phase 1 milestone Mar 18, 2024
@coreyphillips coreyphillips requested a review from ovitrif March 18, 2024 21:39
@coreyphillips coreyphillips self-assigned this Mar 18, 2024
@coreyphillips coreyphillips requested a review from pwltr March 18, 2024 21:50
package.json Outdated Show resolved Hide resolved
@coreyphillips coreyphillips requested a review from pwltr March 19, 2024 11:50
Copy link
Contributor

@ovitrif ovitrif left a comment

Choose a reason for hiding this comment

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

Thank you @coreyphillips for handling this!

Looks good on my end, and still works same as it did when using local dependency pointing to these changes, ie. the data gets migrated on the back, and user-facing everything keeps working, nothing manually needed from the user:

Android Studio 2024-03-19 001639

@coreyphillips coreyphillips merged commit 3909908 into master Mar 19, 2024
4 checks passed
@coreyphillips coreyphillips deleted the chore/upgrade-react-native-keychain branch March 19, 2024 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants