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

Encountered Error during Android Emulation #550

Closed
eugenetan3 opened this issue Jun 27, 2020 · 7 comments
Closed

Encountered Error during Android Emulation #550

eugenetan3 opened this issue Jun 27, 2020 · 7 comments

Comments

@eugenetan3
Copy link

Hi!

I was following #475 and I came across a different error while following the instructions listed there. It's resulting because of an UnhandledPromiseRejectionWarning from CordovaError indicating a Requirements Check for JDK1.8 or greater was failed.
Screen Shot 2020-06-27 at 12 57 40 PM

I believe my iOS emulation should work if I compile in XCode 10.3, because the commandline issue of the "toLowerCase" is resolved via Xcode compilation if I read that correctly.

Something I encounter during iOS build is a reference to a config file AndroidManifest.xml that is not found in my platform/ios folder. Is that intended? I am unfamiliar with Android applications.

Thanks for the help!

@shankari
Copy link
Contributor

shankari commented Jun 27, 2020

@eugenetan3 As you can see from the phone repo README

✨ As part of experimenting with a COVID-19 reference app, we upgraded to the most recent cordova version and added CI ✨ We are now porting this over to the main e-mission repo. The ported code is currently in https://github.com/e-mission/e-mission-phone/tree/upgrade_to_latest_cordova and the current status is in #519. The branch is usable although not all functionality has been tested. Please contribute your testing results so that we feel confident promoting it to master.

I would strongly encourage using that branch instead of master. I could try to debug this for you, but you won't be able to publish the resulting app anyway (#539). I haven't heard back from @tarekeg yet so I assume the branch worked fine :)

@shankari
Copy link
Contributor

If you run into any issues with the branch, please report them at #519
I'm finally getting done with the license updates, and can get to this if there are high priority bugs.

@eugenetan3
Copy link
Author

Awesome thanks, I'll move to that branch and update the issue thread accordingly.

@eugenetan3
Copy link
Author

Quick question, the setup for the branch looks to be a lot easier in terms of installation. Do the same instructions apply previously?

@shankari
Copy link
Contributor

The installation hasn't changed - the branch just automates the installation instructions in a script.

Putting the installation instructions in the script also makes it easier to run CI. Then, if there are issues with the build, people can check the CI to see what to expect.

@shankari
Copy link
Contributor

shankari commented Jun 28, 2020

One caveat, if you installed modules globally (e.g. npm install -g cordova) you might want to uninstall (e.g. npm uninstall -g cordova, similar to https://stackoverflow.com/questions/19523539/apache-cordova-uninstall-globally) before running the setup script. This will ensure that you don't have conflicts with the global and local versions. Of course, if you have other cordova projects, you can keep the global version around as well.

@shankari
Copy link
Contributor

LMK if you have any other questions, otherwise, please close this issue.

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

No branches or pull requests

2 participants