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

[$250] IOS - Build against iOS 15 in ~1 month or the app will be auto-rejected cc #8460

Closed
mvtglobally opened this issue Apr 4, 2022 · 53 comments
Assignees
Labels
Engineering External Added to denote the issue can be worked on by a contributor Improvement Item broken or needs improvement. Weekly KSv2

Comments

@mvtglobally
Copy link

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:

  1. Check IOS app SDK Version

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?

  • iOS

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
Screen Shot 2022-03-24 at 9 46 46 AM

Expensify/Expensify Issue URL:
Issue reported by: @AndrewGable
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1648137460915479

View all open jobs on GitHub

@melvin-bot
Copy link

melvin-bot bot commented Apr 4, 2022

Triggered auto assignment to @danieldoglas (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@melvin-bot melvin-bot bot added the Overdue label Apr 6, 2022
@danieldoglas danieldoglas added External Added to denote the issue can be worked on by a contributor Improvement Item broken or needs improvement. labels Apr 7, 2022
@melvin-bot
Copy link

melvin-bot bot commented Apr 7, 2022

Triggered auto assignment to @jliexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@danieldoglas
Copy link
Contributor

Moved this to external.

@melvin-bot melvin-bot bot removed the Overdue label Apr 7, 2022
@danieldoglas danieldoglas removed their assignment Apr 7, 2022
@jliexpensify
Copy link
Contributor

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Apr 8, 2022
@melvin-bot
Copy link

melvin-bot bot commented Apr 8, 2022

Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (Exported)

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Apr 8, 2022
@melvin-bot
Copy link

melvin-bot bot commented Apr 8, 2022

Triggered auto assignment to @Julesssss (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@melvin-bot melvin-bot bot changed the title IOS - Build against iOS 15 in ~1 month or the app will be auto-rejected cc [$250] IOS - Build against iOS 15 in ~1 month or the app will be auto-rejected cc Apr 8, 2022
@Julesssss
Copy link
Contributor

Awaiting proposals.

@dharmik
Copy link
Contributor

dharmik commented Apr 8, 2022

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
App Store submission requirement starts April 25

  • From who ever's machine we are releasing iOS application we need to verify is Xcode version is 13 or later.
    ==> open Xcode ==> Top menu item click on Xcode ==> Click on About Xcode

  • Version must be 13 or later as I attached in screenshot.

Screen Shot 2022-04-08 at 3 31 08 PM

  • If after updating Xcode and if we want to check iOS SDK version open terminal and write command xcodebuild -showsdks

Screen Shot 2022-04-08 at 3 52 25 PM

Outcome:
If we generate release iOS ipa file will above configuration this email/message ITMS-90725: SDK Version Issue will not come.

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.

@parasharrajat
Copy link
Member

parasharrajat commented Apr 8, 2022

@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.

@dharmik
Copy link
Contributor

dharmik commented Apr 8, 2022

@parasharrajat Are you talking about to change iOS Deployment Target? Please correct me if I misunderstood something.
Screen Shot 2022-04-08 at 4 27 05 PM

If you need someone to do, I will do that.

@parasharrajat
Copy link
Member

parasharrajat commented Apr 8, 2022

Yup, I think that is the one. I don't know much about iOS but that should be it.

@dharmik
Copy link
Contributor

dharmik commented Apr 8, 2022

@parasharrajat
https://user-images.githubusercontent.com/43995119/161472375-1924737e-5479-4f65-b25f-cd8f1fd4d877.png

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

@parasharrajat
Copy link
Member

parasharrajat commented Apr 8, 2022

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

@jliexpensify
Copy link
Contributor

Bump @Julesssss for approval!

I'll be re-assigning this as I am OOO until the 30th.

@parasharrajat
Copy link
Member

Yup. Also, in the PR attach videos for successful build on Xcode 13.

dharmik added a commit to dharmik/App that referenced this issue Apr 20, 2022
@dharmik
Copy link
Contributor

dharmik commented Apr 20, 2022

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

@dharmik
Copy link
Contributor

dharmik commented Apr 20, 2022

Proposal Updated. @AndrewGable

@parasharrajat
Copy link
Member

parasharrajat commented Apr 20, 2022

Do you want me to generate a successful build from the pipeline or from Xcode?

From Xcode(preferably same version as platformDeploy.xml)

@AndrewGable
Copy link
Contributor

Yes, version of XCode built locally must match the code change.

@dharmik
Copy link
Contributor

dharmik commented Apr 20, 2022

@AndrewGable @parasharrajat Is it what we are expecting in video?

Screen.Recording.2022-04-21.at.2.53.04.AM.mov

Mine current xcode version:
Screen Shot 2022-04-21 at 2 54 31 AM

@AndrewGable
Copy link
Contributor

You are going to have to show in video that your changes did not introduce any regressions to the iOS app.

@dharmik
Copy link
Contributor

dharmik commented Apr 21, 2022

@parasharrajat @AndrewGable PR created.
#8738

@AndrewGable
Copy link
Contributor

FYI - Once this is deployed to staging, I can verify that the email is or is not sent via apple.

@dharmik
Copy link
Contributor

dharmik commented Apr 21, 2022

@AndrewGable Sure, Please.

@Julesssss
Copy link
Contributor

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 😆

@dharmik
Copy link
Contributor

dharmik commented Apr 22, 2022

@Julesssss If all well can we merge that PR?

@Julesssss
Copy link
Contributor

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.

@Julesssss
Copy link
Contributor

Merged.

@melvin-bot melvin-bot bot added the Overdue label May 2, 2022
@Julesssss
Copy link
Contributor

Awaiting deploy

@melvin-bot melvin-bot bot removed the Overdue label May 3, 2022
@dharmik
Copy link
Contributor

dharmik commented May 3, 2022

@Julesssss Is there something pending to test for this issue?

@Julesssss
Copy link
Contributor

For now we're awaiting deployment to production

@dharmik
Copy link
Contributor

dharmik commented May 3, 2022

@Julesssss I am new so after this only @jliexpensify can to completed project from upwork?

@Julesssss
Copy link
Contributor

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.

@melvin-bot melvin-bot bot added the Overdue label May 11, 2022
@arielgreen
Copy link
Contributor

Still awaiting deploy

@melvin-bot melvin-bot bot removed the Overdue label May 11, 2022
@arielgreen
Copy link
Contributor

@dharmik paid
@parasharrajat hired for PR review in Upwork, once you accept I'll remit payment

@arielgreen
Copy link
Contributor

Paid.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering External Added to denote the issue can be worked on by a contributor Improvement Item broken or needs improvement. Weekly KSv2
Projects
None yet
Development

No branches or pull requests

9 participants