-
Notifications
You must be signed in to change notification settings - Fork 133
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
Setup Flipper. #43
Setup Flipper. #43
Conversation
Generated by 🚫 dangerJS against f5b84c88be4c52b52ef4cd3ebb09432fa43c49d5 |
f5b84c8
to
d88104f
Compare
|
||
class VectorFlipperProxy( | ||
private val context: Context, | ||
) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ganfra I am not sure that adding this class with alter ego no-op versions is the best way to handle no-op version. WDYT?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd say we have 2 options:
- do not call the initializer if not debug
- isEnabled could also check if it's debug or not ?
This tool does not bring a lot to the project, let's not add it for now. |
Closes #40
This is quite limited, since network on DB are handle by the Rust SDK, so this cannot really be plugged to Flipper. Also there is no Plugin to investigate Jetpack Compose structure (and this will not be supported by the core team, see facebook/flipper#2366).
Can still be useful to take screen shot or video, watch logs, inspect shared preferences and investigate crashes.
This PR set up the tool, so later we may add more plugins.