Skip to content
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

Install Application Permission #107

Closed
wants to merge 3 commits into from
Closed

Install Application Permission #107

wants to merge 3 commits into from

Conversation

slartus
Copy link

@slartus slartus commented Mar 16, 2024

No description provided.

@Alex009
Copy link
Member

Alex009 commented Apr 11, 2024

this permission looks platform specific (it works only for android), so i don't think that we should add this permission in common. users will be misleading by this API

@slartus
Copy link
Author

slartus commented Apr 12, 2024

this permission looks platform specific (it works only for android), so i don't think that we should add this permission in common. users will be misleading by this API

like Permission.STORAGE, Permission.WRITE_STORAGE

@Alex009
Copy link
Member

Alex009 commented Apr 14, 2024

like Permission.STORAGE, Permission.WRITE_STORAGE

in case of this permissions we just not should additionally request permissions for this usecases on ios. but Install of apps is just not exist on iOS platform at all. so when we will see in common code that we request permission to install apps we will think that iOS can install apps someway. but it can't

@slartus
Copy link
Author

slartus commented Apr 14, 2024

like Permission.STORAGE, Permission.WRITE_STORAGE

in case of this permissions we just not should additionally request permissions for this usecases on ios. but Install of apps is just not exist on iOS platform at all. so when we will see in common code that we request permission to install apps we will think that iOS can install apps someway. but it can't

accepted

@slartus slartus closed this Apr 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants