Skip to content

Conversation

@zfurtak
Copy link
Collaborator

@zfurtak zfurtak commented Aug 25, 2025

Details

This PR bumps React Native version in the library and example app to 0.81.0

Related Issues

GH_LINK

Manual Tests

Linked PRs

@zfurtak zfurtak requested a review from Skalakid August 25, 2025 08:33
@zfurtak zfurtak self-assigned this Aug 25, 2025
@Skalakid Skalakid requested a review from WoLewicki September 1, 2025 08:09

const val E_SDK_API = "SDK API Error"
const val E_OPERATION_FAILED = "E_OPERATION_FAILED"
const val E_NO_TOKENS_AVAILABLE = "E_NO_TOKENS_AVAILABLE"
Copy link

Choose a reason for hiding this comment

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

Why was this removed? 🤔

Copy link
Collaborator

Choose a reason for hiding this comment

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

It wasn't used anywhere, so I decided to remove it

@Skalakid Skalakid requested a review from war-in September 2, 2025 13:49
@Skalakid Skalakid marked this pull request as ready for review September 2, 2025 14:25
Skalakid
Skalakid previously approved these changes Sep 3, 2025
Copy link
Collaborator

@WoLewicki WoLewicki left a comment

Choose a reason for hiding this comment

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

Some comments

"react-native": "0.79.2"
"react": "^19.1.0",
"react-native": "0.81.0",
"react-native-safe-area-context": "^5.5.2"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why did you add react-native-safe-area-context in this PR?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

In the example we use SafeAreaView and as it's written it will be deprecated, so I added this dependency to change it

Copy link
Collaborator

Choose a reason for hiding this comment

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

Hmm, I guess it was done automatically when following the upgrade helper

@Skalakid Skalakid requested a review from WoLewicki September 3, 2025 13:49
Copy link
Collaborator

@WoLewicki WoLewicki left a comment

Choose a reason for hiding this comment

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

LGTM!

@Skalakid Skalakid merged commit a5406dd into main Sep 8, 2025
3 checks passed
@os-botify
Copy link
Contributor

os-botify bot commented Sep 8, 2025

🚀 Published to npm in 0.1.9 🎉

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.

5 participants