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

654: remove verification feature flag and add camera permission #657

Merged
merged 1 commit into from
Dec 12, 2022

Conversation

sarahsporck
Copy link
Contributor

No description provided.

@sarahsporck sarahsporck linked an issue Dec 5, 2022 that may be closed by this pull request
@@ -42,4 +42,7 @@
<!--<uses-permission android:name="android.permission.CAMERA" tools:node="remove"/>-->
<uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" tools:node="remove"/>
<uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" tools:node="remove"/>

Copy link
Contributor

Choose a reason for hiding this comment

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

is here a whitespace needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it contributes to a better readability as there is already a space between Location Permission and External Storage Permission

@sarahsporck sarahsporck force-pushed the 654-re-enable-camera-permission branch from f581559 to d9bea88 Compare December 7, 2022 10:07
@sarahsporck sarahsporck enabled auto-merge December 7, 2022 10:07
@sarahsporck sarahsporck force-pushed the 654-re-enable-camera-permission branch 2 times, most recently from 47b3a9c to 45365e9 Compare December 7, 2022 14:01
@sarahsporck sarahsporck force-pushed the 654-re-enable-camera-permission branch from 45365e9 to dbf7c21 Compare December 12, 2022 11:00
@sarahsporck sarahsporck merged commit 2b83b9d into whitelabel Dec 12, 2022
@sarahsporck sarahsporck deleted the 654-re-enable-camera-permission branch December 12, 2022 11:13
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.

Re-enable camera permission
4 participants