Skip to content

[Jetsurvey] Added take a photo feature - #279

Merged
florina-muntenescu merged 10 commits into
dev_alpha07from
fm/camera
Nov 10, 2020
Merged

florina-muntenescu merged 10 commits into
dev_alpha07from
fm/camera

Conversation

@florina-muntenescu

Copy link
Copy Markdown
Contributor

Showcasing launching intents and handling results.

Comment thread Jetsurvey/app/src/main/AndroidManifest.xml Outdated
Comment thread Jetsurvey/app/src/main/res/drawable/ic_selfie_dark.xml Outdated
Comment thread Jetsurvey/app/src/main/res/drawable/ic_selfie_light.xml Outdated

@nickbutcher nickbutcher left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Mostly LG, a few comments.

Comment thread Jetsurvey/app/src/main/java/com/example/compose/jetsurvey/survey/SurveyScreen.kt Outdated
Comment thread Jetsurvey/app/src/main/AndroidManifest.xml Outdated

@yrezgui yrezgui left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Currently the app isn't working on devices running API 28 or less due to the WRITE_EXTERNAL_STORAGE permission. Check my comments on that

Comment thread Jetsurvey/app/src/main/AndroidManifest.xml Outdated
@florina-muntenescu
florina-muntenescu merged commit 8a44a0b into dev_alpha07 Nov 10, 2020
@florina-muntenescu
florina-muntenescu deleted the fm/camera branch November 10, 2020 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants