breautek commented on PR #213: URL: https://github.com/apache/cordova-plugin-device/pull/213#issuecomment-3547903485
> @kumo01GitHub ( cc @breautek ) > > I think we should still prepare a major release before merging this PR in. > > Do you agree? > > When looking at the engine requirements in `plugin.xml` and `package.json` we can see the following cordova-android support: > > ``` > <engine name="cordova-android" version=">=7.0.0" /> > ``` > > I suspect Android 4.4 would be problematic... > > Maybe we should bump major and increase the version requirement to `>=12.0.0`. > > Any thoughts on this? I think you bring up a good point actually. I didn't check cordova-android engine requirements. But yah Android 4.4 is locked to like Chrome 30 something. Android 5 (or 5.1 on some vendors) introduced updatable webviews, but that is limited to around chrome 70ish iirc. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
