Skip to content

Tpi 285 upgrade react#60

Open
JaimeS98 wants to merge 9 commits into
masterfrom
tpi-285-upgrade-react
Open

Tpi 285 upgrade react#60
JaimeS98 wants to merge 9 commits into
masterfrom
tpi-285-upgrade-react

Conversation

@JaimeS98

@JaimeS98 JaimeS98 commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

PR

Upgrade SDK to be compatible with RN v0.86.0 <-> RNv0.78.0
Upgrade Example app to be using RN v0.86.0 as our main target

Testing

Android

Screenshot_20260722_101032_example Screenshot_20260722_101043_Google Play services Screenshot_20260722_101054_example

iOS

20260722_104437 20260722_104443 20260722_104522

Comment thread android/gradle/wrapper/gradle-wrapper.properties
Comment thread android/lib/build.gradle
Comment thread src/tests/test-components/App.tsx Outdated
Comment thread src/@types/jsx.d.ts
@@ -0,0 +1,5 @@
// Global JSX namespace shim for @types/react@19 compatibility.
// React 19 moved JSX to React.JSX; this restores the global JSX.Element alias.
declare namespace JSX {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

If we're setting up JSX as a global alias, is the import in some other specific files in this PR necessary?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Will look if they other imports are unnecessary now, cheers

Comment thread ios/ApplePayButtonManager.swift
Comment thread example/src/App.tsx Outdated
Comment thread example/ios/Podfile
Comment thread ios/TyroPaySdkModule.m
Comment thread Gemfile Outdated
Comment thread package.json Outdated
Comment thread src/tests/WalletPaymentsContainer.spec.tsx Outdated
fix: populate Gemfile.lock checksums after cocoapods upgrade

update readme and e2e script to better select device

update tests, and dependencies plus ci fix for ios

update gemfile lock to point away from vulnerable activesupport versions and update ios-unit ci

update to macoc-15 for unit

create simulator for ios ci tests

update podfile min target and build.gradle to not fail on emtpy tests

remove duplicate ios simulators

fix native unit tests

update e2e tests

make sure to install ios simulator

update e2e github action to try and optimise the flow

add concurrency cancel in progress to unit and dependency

fix prepack order in android e2e

added property check and restored challenge test to android

fix gradle
@JaimeS98
JaimeS98 force-pushed the tpi-285-upgrade-react branch from f1916b0 to c513585 Compare July 21, 2026 01:22
@@ -0,0 +1,4 @@
export enum Colors {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

can we import the other one created? or not possible?

@JaimeS98 JaimeS98 Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yeah can probably just have one actually.

Comment thread example/ios/Podfile
{paths: [process.argv[1]]},
)', __dir__]).strip

MIN_IOS_VERSION='16.0'

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

would've preferred to refer to this variable (wherever it is) and set that in one spot vs hard-coding here.

If we need to bump it up in the future, might be hard to know to do it here too

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

centralised it on package.json

…and fixing some types in tests

podfile.lock update

fix import for min ios version for example podfile
@JaimeS98
JaimeS98 force-pushed the tpi-285-upgrade-react branch from e158dff to 05e5aed Compare July 24, 2026 00:29
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.

4 participants