-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[$250] IOS - Build against iOS 15 in ~1 month or the app will be auto-rejected cc #8460
Comments
Triggered auto assignment to @danieldoglas ( |
Triggered auto assignment to @jliexpensify ( |
Moved this to external. |
Posted: |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat ( |
Triggered auto assignment to @Julesssss ( |
Awaiting proposals. |
Proposal & Information @Julesssss File: .github/workflows/platformDeploy.yml - DEVELOPER_DIR: /Applications/Xcode_12.5.1.app/Contents/Developer
+ DEVELOPER_DIR: /Applications/Xcode_13.2.1.app/Contents/Developer
Main context we have to release iOS application with iOS 15 SDKs Official Information
Outcome: I have Xcode version is 13 or later with latest SDK so let me know if we want to upload ipa file from my machine. Only option we have is just update Xcode version. |
@dharmik I think you misunderstood the issue. Here we wants someone to update the target SDK version on the plist file and corresponding configurations and set up the project so that it will successfully build. Let's me your proposal for doing that. The deployment will be taken care of internally. |
@parasharrajat Are you talking about to change iOS Deployment Target? Please correct me if I misunderstood something. If you need someone to do, I will do that. |
Yup, I think that is the one. I don't know much about iOS but that should be it. |
@parasharrajat We are planning to resolve above informative message which came in email while recently we release application. To resolve this per my proposal we have to stay updated or build application with minimum SDK which is iOS 15 and iOS 15 will be support in Xcode 13 or later so if we generate a build from this configuration it will not give us that email again. CC @Julesssss |
Sounds good. Let's go with that. We have to inform everyone to upgrade their Xcode version. For some of them, it will be hard to do as it may require an OS upgrade. So informing early will be good. cc: @Julesssss 🎀 👀 🎀 C+ reviewed |
Bump @Julesssss for approval! I'll be re-assigning this as I am OOO until the 30th. |
Yup. Also, in the PR attach videos for successful build on Xcode 13. |
Do you want me to generate successful build from pipeline or from Xcode? @parasharrajat |
Proposal Updated. @AndrewGable |
From Xcode(preferably same version as platformDeploy.xml) |
Yes, version of XCode built locally must match the code change. |
@AndrewGable @parasharrajat Is it what we are expecting in video? Screen.Recording.2022-04-21.at.2.53.04.AM.mov |
You are going to have to show in video that your changes did not introduce any regressions to the iOS app. |
@parasharrajat @AndrewGable PR created. |
FYI - Once this is deployed to staging, I can verify that the email is or is not sent via apple. |
@AndrewGable Sure, Please. |
Okay, thanks for the PR. I can't believe I'm saying this, but compiling the app from the newer XCode/iOS sdk worked first time, I was expecting this to be a nightmare to test with all sorts of local compile issues 😆 |
@Julesssss If all well can we merge that PR? |
Yep, just running a few more tests. I used to have two XCode installations (one for the old app, one for the new one), so I'm just confirming all is good after switching to a single app. |
Merged. |
Awaiting deploy |
@Julesssss Is there something pending to test for this issue? |
For now we're awaiting deployment to production |
@Julesssss I am new so after this only @jliexpensify can to completed project from upwork? |
Oh, sorry. Yes, once the issue is deployed to production there will be an automated message here in the issue to let us know you should be paid. |
Still awaiting deploy |
@dharmik paid |
Paid. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
Expected Result:
IOS 15+ SDK should be used for the Expensify app
Actual Result:
IOS 14.5 SDK is used for the Expensify app
Workaround:
unknown
Platform:
Where is this issue occurring?
Version Number: 1.1.44-4

Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @AndrewGable
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1648137460915479
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: