Skip to content
This repository was archived by the owner on May 15, 2024. It is now read-only.

Update Samples.iOS app to be compatible with App Store#1508

Merged
mattleibow merged 12 commits intoxamarin:mainfrom
janusw:issue1467_samples.ios_appstore
Nov 11, 2020
Merged

Update Samples.iOS app to be compatible with App Store#1508
mattleibow merged 12 commits intoxamarin:mainfrom
janusw:issue1467_samples.ios_appstore

Conversation

@janusw
Copy link
Copy Markdown
Contributor

@janusw janusw commented Nov 8, 2020

Description of Change

This PR fixes issue #1467 by dealing with all problems that are reported when trying to upload the Samples.iOS app to App Store Connect, including:

  • several issues with the app icons
  • a problem with an URL scheme (related to App Center)
  • deprecated API usage (UIWebView)

Bugs Fixed

API Changes

None.

Behavioral Changes

None.

PR Checklist

  • Has tests (if omitted, state reason in description)
  • Has samples (if omitted, state reason in description)
  • Rebased on top of main at time of PR
  • Changes adhere to coding standard
  • Updated documentation (see walkthrough)

janusw added 10 commits November 8, 2020 13:32
* required by App Store Connect
* App Store Connect complained:

ERROR ITMS-90158: "The following URL schemes found in your app are not in the correct format: [appcenter-APP_SECRET]. URL schemes need to begin with an alphabetic character, and be comprised of alphanumeric characters, the period, the hyphen or the plus sign only. Please see RFC1738 for more detail."
* required by App Store Connect
* required by App Store Connect
* exported from the .sketch file
Copy link
Copy Markdown
Member

@mattleibow mattleibow left a comment

Choose a reason for hiding this comment

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

Looks good. Maybe we can keep the same version numbers of "1.0" so we don't have to update it for each release.

Also, I see a comment in the issue about the CFBundleIconName. And, where are the "several cases of "Invalid Image Path"" issues resolved?

@mattleibow mattleibow added this to the 1.6.0 milestone Nov 8, 2020
@janusw
Copy link
Copy Markdown
Contributor Author

janusw commented Nov 9, 2020

Looks good. Maybe we can keep the same version numbers of "1.0" so we don't have to update it for each release.

Yeah, well, that is kind of optional from my side. I thought it would be better to update it to the current version now, so that it's somewhat less outdated, even if it will not be updated for every release.

Also, I see a comment in the issue about the CFBundleIconName.

I think that was fixed by the migration to the asset catalog (see 544c970), which removed the CFBundleIconFiles.

And, where are the "several cases of "Invalid Image Path"" issues resolved?

In 4166093. Info.plist contained a few references to icons that don't exit.

@mattleibow mattleibow merged commit 2e0ec19 into xamarin:main Nov 11, 2020
@janusw janusw deleted the issue1467_samples.ios_appstore branch November 12, 2020 20:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Samples.iOS is not fit for TestFlight / App Store

2 participants