Permissions Added #10
Replies: 6 comments
-
I actually dont think it should be in the core. I think N should be splitted as much as possible. I dont see any good reason to put it "in". Just like I think the ui should be splitted into a separated plugin. |
Beta Was this translation helpful? Give feedback.
This comment was marked as disruptive content.
This comment was marked as disruptive content.
-
Just One thing please @NathanaelA stop with your download number thing you keep on bringing it up everywhere. It makes no sense. Of course a plugin there for so much more time will have bigger number. And we might want to keep one in the community. Much eqisier for us to maintain and release as we use it many community plugins already. We dont have to wait for N to release a fix or something when we need a change we can simply release. So choose as you prefer I ll support whatever the choice is as long as it stays outside of N ;) |
Beta Was this translation helpful? Give feedback.
This comment was marked as disruptive content.
This comment was marked as disruptive content.
-
My two cents: From what you are communicating you're planning to extend the scope of NativeScript, e.g. with #11 . So while more platforms are being added the NS team would have to make sure the core permissions work for every platform. This will result in additional development and maintainance efforts. If permissions were available as a plugin we could simply use your plugin for iOS and Android and use another one for another platform without it feeling "wrong".
Additionally, there are a few apps that don't require any sort of permission management. Don't know if integrated core permissions would be disadvantageous in any form for those apps. |
Beta Was this translation helpful? Give feedback.
-
I think NS 8.0 should have the NS-Permissions plugin built in. The Permissions plugin is the most widely used plugin in the entire eco-system, and even used by some of the NS standard plugins. Seems to make sense to integrate it into the core so that it is no longer a pretty much required "add-on".
I don't know if we should work on making the API a bit more generic to handle IOS as the current plugin is Android only, but it a thought to allow both the current method and adding helper features to support IOS & Android permissions like
Permission.get(CAMERA).then(...) where it asks for both Android and IOS permissions at the same time?
Beta Was this translation helpful? Give feedback.
All reactions