-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Feature Request: Programmatic API to launch LeakActivity #1337
Comments
Would it be also useful to have API to start leak detection/analysis? Since we have a threshold to launch analysis I tend to tap back and force to trigger it while checking if I fixed it or not. |
@emartynov can you file a separate issue? @ZacSweers I'm hesitating between starting the activity and producing an intent that will. The later is more flexible but requires a context and more code, the former is straightforward but couldn't be used for say pending intents or adding shortcuts etc. |
Also worth noting LeakCanary 2 adds a dynamic shortcut on the main app launcher |
I'd be fine with just creating an intent too!
…On Tue, May 7, 2019 at 1:51 AM PY ***@***.***> wrote:
@emartynov </~https://github.com/emartynov> can you file a separate issue?
@ZacSweers </~https://github.com/ZacSweers> I'm hesitating between starting
the activity and producing an intent that will. The later is more flexible
but requires a context and more code, the former is straightforward but
couldn't be used for say pending intents or adding shortcuts etc.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1337 (comment)>,
or mute the thread
</~https://github.com/notifications/unsubscribe-auth/AAKMJPWK5QT42NYCESJ2DLLPUE7KXANCNFSM4HLGOY3Q>
.
|
|
Resolves #1337 Kept it as a function rather than a val since the best we could do due to the context requirement would be an awkward `lateinit var`.
This is something I think would be useful for debug builds (especially for companies with multiple apps) to allow for launching the leak activity directly (via button in a debug drawer or something similar)
Something like this:
LeakCanary.startLeakActivity(context)
The text was updated successfully, but these errors were encountered: