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

Dvci 265 icon splash #53

Merged
merged 5 commits into from
May 9, 2022
Merged

Dvci 265 icon splash #53

merged 5 commits into from
May 9, 2022

Conversation

nkarip
Copy link
Contributor

@nkarip nkarip commented Apr 27, 2022

Add default app icon, disable splash screen, fix suspense fallback

Copy link
Contributor

@sarahmcdougall sarahmcdougall 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 on android, but I am having trouble building with iOS. I am getting the following error:

The following build commands failed:
	CompileAssetCatalog /Users/smcdougall/Library/Developer/Xcode/DerivedData/SMART_Health_Card_Verifier-butwbgdonuwimeesisigtlithivu/Build/Intermediates.noindex/ArchiveIntermediates/SMART\ Health\ Card\ Verifier/InstallationBuildProductsLocation/Applications/SMART\ Health\ Card\ Verifier.app /Users/smcdougall/vci/shc-web-verifier/platforms/ios/SMART\ Health\ Card\ Verifier/Images.xcassets (in target 'SMART Health Card Verifier' from project 'SMART Health Card Verifier')
(1 failure)
xcodebuild: Command failed with exit code 65

Any ideas on how to get the Cordova build to succeed for iOS? I think it's due to the splash image but could not come up with any fixes.

@nkarip
Copy link
Contributor Author

nkarip commented May 2, 2022

Turns out ios does not allow single-icon configuration, see apache/cordova-docs#1145. I changed to use an Ionic utility to generate the required icon files and update the config.xml. See README for instructions. Tested on ios simulator.

@sarahmcdougall sarahmcdougall self-requested a review May 4, 2022 13:00
Copy link
Contributor

@sarahmcdougall sarahmcdougall left a comment

Choose a reason for hiding this comment

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

Cordova is able to build for iOS now! I tested it both on device and on simulator. I did notice that the icon shows up now but its more enlarged on iOS than on android, so it gets cut off a little. Also, on android I am able to see the "Loading..." paragraph on the splash page, but that does not appear on iOS. Not sure if these observations should be addressed in this PR - let me know your thoughts.

Screen Shot 2022-05-04 at 9 01 41 AM

@nkarip
Copy link
Contributor Author

nkarip commented May 4, 2022

I think the icon looking funny is ok, since we're not releasing to the app store. There is a separate ticket for designing a real icon if we want to in the future. The underlying code/config would not change.
The "Loading..." view is not the mobile app splash page, it's the React suspense fallback page and happens on webapp as well, so not mobile specific. The original mobile app splash page was the Cordova default which showed the Cordova robot logo. This is now disabled.

@sarahmcdougall sarahmcdougall self-requested a review May 9, 2022 17:32
Copy link
Contributor

@sarahmcdougall sarahmcdougall left a comment

Choose a reason for hiding this comment

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

Since we aren't worried about the app icon right now and cordova can build on both iOS and Android, this is good to merge!

@sarahmcdougall sarahmcdougall merged commit 18384da into main May 9, 2022
@sarahmcdougall sarahmcdougall deleted the dvci-265-icon-splash branch May 9, 2022 17:38
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.

2 participants