Skip to content
This repository has been archived by the owner. It is now read-only.

Denied status and Camera/Microphone/Photos added #8

Merged
merged 12 commits into from Apr 24, 2015
Merged

Denied status and Camera/Microphone/Photos added #8

merged 12 commits into from Apr 24, 2015

Conversation

@bre7
Copy link
Collaborator

@bre7 bre7 commented Apr 23, 2015

Added new button style (same as Authorized, inverse tint color) and title for DENIED permissions, alert is also shown asking the user to go to settings....) (#2 ?)

Camera/Microphone permissions added (#1).
Bug: Microphone isn't working 100%. Using the test project, it says "ALLOWED MICROPHONE" when it should say "ALLOW" since the permissions where never granted before.

@nickoneill
Copy link
Owner

@nickoneill nickoneill commented Apr 23, 2015

This is great, thanks! I'll check it out this afternoon.

@bre7
Copy link
Collaborator Author

@bre7 bre7 commented Apr 23, 2015

Photos added. (#3)

Bug: When Photos is requested, Microphone is taken into account. The button title says "Photos" but the triggered alert doesn't: "Permission for Microphone was denied." (No mixup like before, not sure why it's happening)
statusPhotos() says Authorized but PHPhotoLibrary.authorizationStatus() states .NotDetermined...was working on an old verion

@nickoneill nickoneill changed the title Denied status and Camera/Microphone added Denied status and Camera/Microphone/Photos added Apr 23, 2015
@nickoneill nickoneill merged commit f16eed5 into nickoneill:master Apr 24, 2015
@nickoneill
Copy link
Owner

@nickoneill nickoneill commented Apr 24, 2015

Thank you! I made a couple fixes that prevented the denied alert from showing, everything else was excellent though.

@bre7
Copy link
Collaborator Author

@bre7 bre7 commented Apr 24, 2015

Np, thanks for the fixes

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants