Skip to content

Cesar/version 2025.6.1#53

Merged
cesar-sosa-hol merged 4 commits intomainfrom
cesar/version-2025.6.1
Jun 5, 2025
Merged

Cesar/version 2025.6.1#53
cesar-sosa-hol merged 4 commits intomainfrom
cesar/version-2025.6.1

Conversation

@cesar-sosa-hol
Copy link
Collaborator

update version of RN Sdk
Fix iOS upgrade
update readme

@cesar-sosa-hol cesar-sosa-hol requested a review from Armaxis June 5, 2025 05:46
)', __dir__]).strip

platform :ios, 15.1
platform :ios, 16
Copy link
Member

@Armaxis Armaxis Jun 5, 2025

Choose a reason for hiding this comment

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

Someone from iOS team should confirm this

Choose a reason for hiding this comment

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

yes, we bumped the version to 16.0

Armaxis
Armaxis previously approved these changes Jun 5, 2025
@jyotikdev
Copy link

@cesar-sosa-hol did you test iOS? processingMode is now required in the startPayment

@cesar-sosa-hol
Copy link
Collaborator Author

image
Yes, startPayment is working, even i completed a payment

@cesar-sosa-hol
Copy link
Collaborator Author

@cesar-sosa-hol did you test iOS? processingMode is now required in the startPayment

i forgot push a mapper change for ios


guard let processingModeInt = paymentParameters["processingMode"] as? Int,
let processingMode = ProcessingMode(rawValue: processingModeInt) else {
return .failure(.missingIdempotencyKey)

Choose a reason for hiding this comment

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

this should say missingProcessingMode

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, Thanks, @jyotikdev please check it again

@cesar-sosa-hol cesar-sosa-hol requested a review from jyotikdev June 5, 2025 16:10
@cesar-sosa-hol cesar-sosa-hol merged commit 9649c60 into main Jun 5, 2025
2 of 5 checks passed
@cesar-sosa-hol cesar-sosa-hol linked an issue Jun 5, 2025 that may be closed by this pull request
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.

Pod Install Uses Definition in package.json

3 participants