-
Notifications
You must be signed in to change notification settings - Fork 63
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
Manifest merger failed when using v5.0.0-beta.1 / permission error on Android <=10 #96
Comments
Short update: I found out that I had to remove following part from the manifest:
This fixed the merge problem and the project builds. Plugin works now fine on Android 9 and 11, but on Android 7 and 8 the Plugin opens and I can take a photo. Then the App crashes with this permission error:
@ChrisTomAlx Any ideas? |
Another update. After some more testing i noticed following: Android <=9 device: App crashes and restart when switching from camera view to image cropper view. So far I wasn't able to check the error. My solution so far: Open Android App Settings, go to Permissions. The 'Storage' permission is marked as declined, but I was never asked for granting access to storage. When setting this manually to accept, the Plugin works fine on all Devices with Android <=10. Is there any way to request permissions for storage in a propper way? |
@SchmidtKBS sorry about the delay, you could try and use the diagnostic plugin to request for permissions |
Hey @ChrisTomAlx, I managed to get it running. I didn't use the diagnostic plugin. I made a small 'fix' to the scanlibrary and i got it running on Android <=9. You can check these to commits on my fork: first, second I also noticed another problem, when using the plugin the first time. After opening the plugin for the first time, it asks for camera permissions. After granting permissions the library stays in PickImageFragment instead of redirectiong to camera. I found a fix for this to get the code working as intended, but this requires at least Android 6: click Best regards, |
Hi, I faced this issue too after implement this plugin.
The plugin work properly on Android 11 (target API 29). I tried to add back the provider to AndroidManifest, I could solve the merge problem with this modification:
After that the Capacitor Camera plugin works again, but this Document scanner crashed the app on call .scanDoc(). (Tested on Android 11),
|
Any update on this? @ChrisTomAlx Cheers, |
Still facing this issue |
@hembachrterran did you find a workaround for using Capacitor Camera and this plugin in the same app? |
@rkreutzer Sorry, but no. The problem is with this package. We skipped this feature. I have no time to make a PR. @ChrisTomAlx Any update on this problem? Can you provide a solution? |
Sorry don't have the time to look at this at the moment @hembachrterran. But always welcoming PR's. EDIT - Apologies accidentally closed the issue. |
Hello, did anyone got a solution? Thank you so much. |
As a workaround for now, I removed the Capacitor/camera and installed cordova-plugin-camera. I removed the section from the AndroidManifest file and the camera now functions with this plug-in. |
@SchmidtKBS how can I apply your changes to the scanlibary in my project? Thank you! |
Hello, I did a update from plugin version 4.2.5 to 5.0.0-beta.1. After opening the android folder of my project in Android Studio I tried to run a build, but it failed because of an error during the manifest merging:
My xml looks like this:
It tried to follow the suggestions made by Android Studio. This fixed the build, but when I run the App (on Android 9 Device and Pixel 2 Emulator) it crashes when I open the ScannerPlugin:
Any ideas, what is wrong here?
The text was updated successfully, but these errors were encountered: