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

Upgrade IOS version to 1.7.2 #112

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mansi-square
Copy link
Collaborator

  1. Upgrade IOS version to 1.7.2
  2. Upgrade app plugin version

1. Upgrade IOS version to 1.7.2
2. Upgrade app plugin version
@CLAassistant
Copy link

CLAassistant commented Apr 22, 2024

CLA assistant check
All committers have signed the CLA.

@@ -1,5 +1,9 @@
## Changelog

### v6.0.0 Apr 22st, 2024

Choose a reason for hiding this comment

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

Let's just put "Apr 22" here. "Apr 22st" is not correct (same applies to line 7 - can you update this as well?)

@@ -144,6 +144,7 @@
3B06AD1E1E4923F5004D2608 /* Thin Binary */,
17130ABE28ACDC4A007FE54B /* Square setup script */,
324E410A8DA310037B0B821B /* [CP] Embed Pods Frameworks */,
CB238059B6C64501CA7FAC41 /* [CP] Copy Pods Resources */,
Copy link

@tobischw tobischw Apr 22, 2024

Choose a reason for hiding this comment

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

I am not totally sure we want this at the end (does the setup script fail?) Need to check with @brandonjenniges on his thoughts. I think it's OK since it's just resources and I don't think the setups script is a resource?

Copy link
Member

Choose a reason for hiding this comment

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

The Square setup script needs to be after both [CP] Embed Pods Frameworks and [CP] Copy Pods Resources

Choose a reason for hiding this comment

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

Thanks for confirming!

@@ -3,7 +3,7 @@
#
Pod::Spec.new do |s|
s.name = 'square_reader_sdk'
s.version = '3.0.0'

Choose a reason for hiding this comment

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

I am curious why this was 3.0.0 before?

Copy link
Member

Choose a reason for hiding this comment

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

Looks like before it was attempting to be synced with this

.

I think this podspec is never published to CocoaPods so I think technically we can down version it like this.. but maybe we won't to keep it in sync with the plugin version? So 5.0.0, it looks like is what we're calling this next version.

@@ -1,5 +1,5 @@
# Uncomment this line to define a global platform for your project
platform :ios, '11.1'
platform :ios, '14.0'
Copy link
Member

Choose a reason for hiding this comment

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

When trying to run the example app but getting some errors with deployement version and I see some other places that appear to need to update the target to 14.

square_reader_sdk.podspec's s.ios.deployment_target
Runner.xcodeproj/project.pbxproj is still referencing ios 11

@brandonjenniges
Copy link
Member

I think this version number needs an update also.

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.

None yet

4 participants