[GitHub] cordova-lib issue #492: CB-11908 Add edit-config to config.xml

Github user andreszs commented on the issue: https://github.com/apache/cordova-lib/pull/492 @dpogue That worked perfectly, thank you!! +10 --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For addition

[GitHub] cordova-lib issue #492: CB-11908 Add edit-config to config.xml

Github user dpogue commented on the issue: https://github.com/apache/cordova-lib/pull/492 @andreszs Something like this should work: ```xml ``` --- - To unsubscribe, e-mail: dev-unsubs

[GitHub] cordova-lib issue #492: CB-11908 Add edit-config to config.xml

Github user andreszs commented on the issue: https://github.com/apache/cordova-lib/pull/492 @ktop Please let me know if the plugin can be used to add the attribute `android:installLocation="auto"` into the `manifest` node itself. The docs only explain how to add **child nodes**, and I

[GitHub] cordova-lib issue #492: CB-11908 Add edit-config to config.xml

Github user stevengill commented on the issue: https://github.com/apache/cordova-lib/pull/492 Thanks for sending this pr @ktop! I've merged it in. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] cordova-lib issue #492: CB-11908 Add edit-config to config.xml

Github user stevengill commented on the issue: https://github.com/apache/cordova-lib/pull/492 I'm going to merge this in for now so I can get the common release out. But it is definitely worth discussing some of these points. --- If your project is set up for it, you can reply to th

[GitHub] cordova-lib issue #492: CB-11908 Add edit-config to config.xml

Github user stevengill commented on the issue: https://github.com/apache/cordova-lib/pull/492 `remove` wouldn't be a revert right. Since you have no way to store what got overwritten/merged --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] cordova-lib issue #492: CB-11908 Add edit-config to config.xml

Github user stevengill commented on the issue: https://github.com/apache/cordova-lib/pull/492 do you have a usecase example when remove might be useful? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] cordova-lib issue #492: CB-11908 Add edit-config to config.xml

Github user stevengill commented on the issue: https://github.com/apache/cordova-lib/pull/492 @ktop so if I remove an `edit-config` tag from my `config.xml`, it won't actually remove it from `AndroidManifest.xml`. Guessing the solution for users in this case would be to rm and re-add

[GitHub] cordova-lib issue #492: CB-11908 Add edit-config to config.xml

Github user codecov-io commented on the issue: https://github.com/apache/cordova-lib/pull/492 ## [Current coverage](https://codecov.io/gh/apache/cordova-lib/pull/492?src=pr) is 80.38% (diff: 100%) > Merging [#492](https://codecov.io/gh/apache/cordova-lib/pull/492?src=pr) into [ma

[GitHub] cordova-lib issue #492: CB-11908 Add edit-config to config.xml

Github user ktop commented on the issue: https://github.com/apache/cordova-lib/pull/492 Thanks, I fixed the errors. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and w

[GitHub] cordova-lib issue #492: CB-11908 Add edit-config to config.xml

Github user stevengill commented on the issue: https://github.com/apache/cordova-lib/pull/492 JSHint is failing for you --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled an