Skip to content
This repository has been archived by the owner on Jan 11, 2024. It is now read-only.

FIX-FIN203 #80

Closed
wants to merge 3 commits into from
Closed

FIX-FIN203 #80

wants to merge 3 commits into from

Conversation

garvit984
Copy link

Fixes #78 FIN-203

Checking for Camera Permission instead of Write Permission in CustomerProfileActivity

Click on "Customer" inside the drawer.
Click on any item in the list of customers.
Click on the profile picture (by default it's the Mifos Logo)
Click on the "Share" option in the app bar.
Supposing the permission to write to external storage is not given, it'll ask for permission. When granted it should ideally call the "share image" method, but nothing happens. This is because when the Permission is being granted, it's checking for "CAMERA" permission instead of "WRITE_EXTERNAL_STORAGE" permission.
It occurs when the permission is being granted initially and later, this error does not occur.
Please make sure these boxes are checked before submitting your pull request - thanks!

  • Apply the AndroidStyle.xml style template to your code in Android Studio.

  • Run the unit tests with ./gradlew check to make sure you didn't break anything.

  • If you have multiple commits please combine them into one commit by squashing them.

@garvit984 garvit984 closed this Mar 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant