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

Fix user agent match issue android pie #535

Conversation

pradiv-kumar
Copy link

Platforms affected

Android

What does this PR do?

Fix the user agent match issue in Android Pie devices #534

What testing has been done on this change?

Checklist

@janpio janpio added the bug label Nov 5, 2018
@codecov-io
Copy link

Codecov Report

Merging #535 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #535   +/-   ##
=======================================
  Coverage   61.87%   61.87%           
=======================================
  Files          17       17           
  Lines        1975     1975           
  Branches      366      366           
=======================================
  Hits         1222     1222           
  Misses        753      753

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e58453d...cf2dd6f. Read the comment docs.

@brody4hire
Copy link

Thanks for the contribution. Considering that check is only used to check for version range of 4.0...4.3 and minimum Android version is now 4.4 ([1]) I would suggest a much simpler solution: simply remove the lines that check check and version_code, along with some of the comments above.

Another note is that we generally use the cordova-coho tool to generate bin/templates/project/assets/www/cordova.js. I would prefer if that you remove the change to bin/templates/project/assets/www/cordova.js from this PR.

[1] https://cordova.apache.org/docs/en/latest/guide/platforms/android/index.html#requirements-and-support

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

Successfully merging this pull request may close these issues.

4 participants