-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
chore(cordova/android): upgrade min target android API to 23 #1699
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #1699 +/- ##
=======================================
+ Coverage 36% 45% +8%
=======================================
Files 45 39 -6
Lines 2749 1943 -806
Branches 312 312
=======================================
- Hits 1006 885 -121
+ Misses 1743 1058 -685
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -23,7 +23,7 @@ | |||
<access origin="*" /> | |||
<allow-intent href="http://*/*" /> | |||
<allow-intent href="https://*/*" /> | |||
<preference name="android-minSdkVersion" value="22" /> | |||
<preference name="android-minSdkVersion" value="23" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need to upgrade, and what are the implications in user numbers?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's required by the Play Store!
Are you asking because you want it to be documented on this PR, or because you forgot about our email correspondence?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Too many things to track. I found the details, the deadline is August 31, 2023, so we have to do it anyway.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for taking care of this!
No description provided.