galgo icon indicating copy to clipboard operation
galgo copied to clipboard

system-overlay-window again on 6.0.1

Open erikswed opened this issue 7 years ago • 6 comments

And Android 6.0.1 api23 When I add this

compile 'com.github.inaka:galgo:v1.0.2'

The app crash with:

Caused by: java.lang.IllegalStateException: in order to use Galgo, please add the permission android.permission.SYSTEM_ALERT_WINDOW to your AndroidManifest.xml.

Even do I have this perrmission and also it´s included in galgo lib right

I cannot figure out this please advice?

I read that I should test for this permission since I side-lode the apk

https://stackoverflow.com/questions/32652533/android-system-overlay-window

but app crash before Activity onCreate

erikswed avatar Aug 01 '17 19:08 erikswed

also seeing on 6.0.1 and 7.1.2, but not on 5.1.1, perhaps M permissions handling hasn't been added yet?

SaikCaskey avatar Aug 03 '17 10:08 SaikCaskey

I've submitted a PR with changes that allow this to work for you

SaikCaskey avatar Aug 03 '17 12:08 SaikCaskey

It's number #28 (just to link it here).

elbrujohalcon avatar Aug 03 '17 12:08 elbrujohalcon

Grate Thanks when will you release v2.0.0

erikswed avatar Aug 03 '17 13:08 erikswed

yo he never accepted it or did the changes himself ☹️

my fork still works for O, haven't tested P yet, will likely do so soon. implementation 'com.github.saikcaskey:systemecks-galgo:1.0.1'

please just update this library tho, your CI gadget was wrong to reject my PR

SaikCaskey avatar Jul 19 '18 13:07 SaikCaskey

Hey @tenmaster @griveroa @quiqueqs … anything to comment here?

elbrujohalcon avatar Jul 19 '18 13:07 elbrujohalcon