add limitsNavigationsToAppBoundDomains to configuration

apache / cordova-ios

Apache Cordova iOS

https://cordova.apache.org/

Apache License 2.0

2.15k stars 987 forks source link

add limitsNavigationsToAppBoundDomains to configuration #1325

Closed clearisoft closed 1 year ago

clearisoft commented 1 year ago

Platforms affected

Motivation and Context

Description

Testing

Checklist

[ ] I've run the tests to see all new and existing tests pass
[ ] I added automated test coverage as appropriate for this change
[ ] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
[ ] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
[ ] I've updated the documentation if necessary

breautek commented 1 year ago

If applicable, the PR should point to the master branch so that versions moving forward will contain the change. Cordova doesn't really do backports outside of extraordinary circumstances, but if someone is willing to do a backport, they can cherry-pick the PR into backport feature branch.

dpogue commented 1 year ago

This is a duplicate of https://github.com/apache/cordova-ios/pull/1249 except defaulting to YES (which I don't think we want to do because that will break existing apps).

dpogue commented 1 year ago

Closing this because https://github.com/apache/cordova-ios/pull/1249 has been merged as part of the 7.0.0 release.