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

fix(docs/camera): add usage descriptions for using in iOS #2633

Merged
merged 1 commit into from
Mar 25, 2020

Conversation

rdlabo
Copy link
Contributor

@rdlabo rdlabo commented Mar 25, 2020

Capacitor's camera plugin is require three permission: https://github.com/ionic-team/capacitor/blob/master/ios/Capacitor/Capacitor/Plugins/Camera.swift#L348-L373

But now, docs said require one permission. This pull request is fix this.
Thanks.

Copy link
Member

@jcesarmobile jcesarmobile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

But for the ionic camera app, as it only uses the camera and not the photo library nor saves to it, they are not needed there, so to keep the guide simpler, better don't add unnecessary things.

@@ -218,7 +218,7 @@ Next, for the Camera plugin to work, we must configure the "Privacy - Camera Usa

Each setting in `Info.plist` has a low-level parameter name and a high-level name. By default, the property list editor shows the high-level names, but it's often useful to switch to showing the raw, low-level names. To do this, right-click anywhere in the property list editor and toggle "Show Raw Keys/Values."

Locate the `NSCameraUsageDescription` Key (or add it) and set the Value to something that describes why the app needs to use the camera, such as "To Take Photos." The Value will be displayed to the app user when the permission prompt opens.
Locate the `NSCameraUsageDescription` Key (or add it) and set the Value to something that describes why the app needs to use the camera, such as "To Take Photos." The Value will be displayed to the app user when the permission prompt opens. Do the same for `NSPhotoLibraryAddUsageDescription` Key and `NSPhotoLibraryUsageDescription` Key.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The sample apps doesn't read or save to the gallery, so those two are not needed for the guide.

@rdlabo
Copy link
Contributor Author

rdlabo commented Mar 25, 2020

@jcesarmobile Thanks for review. I removed tutorial change. thanks.

Copy link
Member

@jcesarmobile jcesarmobile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, thanks!

@jcesarmobile jcesarmobile changed the title fix(docs/camera): add permission for using in iOS fix(docs/camera): add usage descriptions for using in iOS Mar 25, 2020
@jcesarmobile jcesarmobile merged commit 7692ee4 into ionic-team:master Mar 25, 2020
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