-
Notifications
You must be signed in to change notification settings - Fork 140
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
minify support for the library? #68
Comments
Hi @mihir-kandoi , |
Thank you :) |
Is there a fix for this? Our minified app also fails. |
If someone still needs this, try with this:
|
Thank you mate. I no longer need it but it will be super helpful for
someone who does! @devs please bake this!
…On Sun, Jul 9, 2023 at 8:28 PM Alexander Štimac ***@***.***> wrote:
If someone still needs this, try with this:
-dontwarn com.espressif.provisioning.**
-dontwarn espressif.**
-keep class com.espressif.provisioning.** {*;}
-keepclassmembers class com.espressif.provisioning.** {*;}
-keep class espressif.** {*;}
-keepclassmembers class espressif.** {*;}
-keep interface com.espressif.provisioning.** {*;}
-keep interface espressif.** {*;}
-keep public enum com.espressif.provisioning.** {*;}
-keep public enum espressif.** {*;}
—
Reply to this email directly, view it on GitHub
<#68 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACDMRNXGAFO6GT7ITLYUFSLXPLBIFANCNFSM57ZQNFCA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hi @astimac , thank you. |
Thanks @astimac - works like a charm 👍 |
When minify is enabled in build.gradle, everything seems fine and the app builds but the provisioning library does not work correctly (for ex. scanning networks method simply does not return). Disabling minify fixes the issue BUT just because of this single library we can't use minify which when enabled cuts down our app size to half. I also see the example app is also not using minify. I request the developers to try with minify themselves and update the library with embedded ProGuard rules.
The text was updated successfully, but these errors were encountered: