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

hello-world app should be submittable at the AppStore #1490

Closed
PanayotCankov opened this issue Feb 16, 2016 · 9 comments
Closed

hello-world app should be submittable at the AppStore #1490

PanayotCankov opened this issue Feb 16, 2016 · 9 comments

Comments

@PanayotCankov
Copy link
Contributor

Currently submission to the AppStore throws the following errors:

ERROR ITMS-90022: Missing required icon file. The bundle does not contain an app icon for iPhone / iPad Touch of exactly '120x120' pixels, in .png format for iOS versions >= 7.0."
ERROR ITMS-90023: Missing required icon file. The bundle does not contain an app icon for iPad of exactly '76x76' pixels, in .png format for iOS versions .+ 7.0.
ERROR ITMS-90023: Missing required icon file. The bundle does not contain an app icon for iPad of exactly '152x152' pixels, in .png format for iOS versions >= 7.0.
ERROR ITMS-90023: Missing required icon file. The bundle does not contain an app icon for iPad of exactly '167x167' pixels, in .png format.
ERROR ITMS-90476: Invalid Bundle. iPad Multitasking support requires launch story board file 'LaunchScreen' in bundle 'org.nativescript.TNSAppTests'.

The icons may need to be listed in the Info.plist or carefully stored in xcasset (#683), there is a similar issue (#888).
The LaunchScreen was not added to my app's target.

@rosen-vladimirov
Copy link
Contributor

I believe the missing icons should be added in the default App_Resources (tns-template-hellow-world). CLI cannot do anything in case you just delete some of them.

@rosen-vladimirov
Copy link
Contributor

Maybe I'm wrong, but as far as I remember, the LaunchScreen should be added in the default Info.plist. So this is again part of the tns-template-hello-world and in case the user modifies it, CLI has nothing to do about it.
@PanayotCankov why do you consider this as bug in CLI, maybe I'm missing something here?

@fealebenpae
Copy link
Contributor

@PanayotCankov @rosen-vladimirov the default Info.plist should explicitly disable iPad multitasking which will obviate the need for a LaunchScreen. iPad multitasking relies on AutoLayout and size classes so it's likely not to work for apps using NativeScript layouts anyway.

@PanayotCankov
Copy link
Contributor Author

We will have to update the Info.plist in the hello world templates:

List all icons from the template in the Info.plist
CFBundleIconFiles
https://github.com/NativeScript/nativescript-marketplace-demo/blob/production/app/App_Resources/iOS/Info.plist#L43

Switch off iPad multitasking requiring
UIRequiresFullScreen
https://github.com/NativeScript/nativescript-marketplace-demo/blob/production/app/App_Resources/iOS/Info.plist#L31

We may also switch off the LaunchScreen storyboard and add all variants of the launch images. I think we currently miss some for iPhone6, 6+ etc.

@PanayotCankov
Copy link
Contributor Author

We can also use Images.xcassets.

The CLI will add app/App_Resources/iOS/Images.xcassets as resource to the Xcode project.
In the app/App_Resources/iOS/build.xcconfig the following two flags can set icon and launch image from the Images.xcassets:

ASSETCATALOG_COMPILER_APPICON_NAME = AppIcon;
ASSETCATALOG_COMPILER_LAUNCHIMAGE_NAME = Brand Assets;

@fealebenpae
Copy link
Contributor

iPad Pro will only support launch storyboards, not images: https://openradar.appspot.com/22915712

@PanayotCankov
Copy link
Contributor Author

@fealebenpae Launch Storybards are better than images. But then it is hard to provide a decent editor outside the Xcode project. IMO we should have used a full Xcode project in the template (~app/platform) and inserted the resources and run-times similar to the way cocoa pods adds dependencies.

The best thing I can imagine for a launch storyboard, that is editable by a non-Xcode user, would be a 9-patch-like image, displayed by our default launch storyboard.

How would you address this issue?

@fealebenpae
Copy link
Contributor

@PanayotCankov Place a simple storyboard in App_Resources with a single image view that loads the LaunchImage asset. This will work for 99% of users, IMO, the rest would need to edit the storyboard.

@jumenta1960
Copy link

Buenos dias. Por favor tengo el error 90023 icons. Que debo hacer en Info.plist? Gracias @AppColombiaNet
unspecified Lo anterior para poder subir el archivo a Tunes. Gracias

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants