-
Notifications
You must be signed in to change notification settings - Fork 127
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
RFC: Decoupling Flipper from React Native core #641
RFC: Decoupling Flipper from React Native core #641
Conversation
876ca21
to
7012ae0
Compare
7012ae0
to
6be3baa
Compare
Looks good from a react-native-windows perspective. |
Good from RN CLI point of view. Since we're essentially losing all the non-js debugging tools working out-of-the-box with Flipper, I'd put emphasis on how to integrate Flipper again to one's codebase. I believe this part is missing in the RFC – what would be the actions that a user needs to take to integrate Flipper after e.g. installing this new package. |
6be3baa
to
a1a6d3a
Compare
Does this only include Also this got me thinking for extensibility option. Would we be able to install custom chrome extensions? Like leverage existing React DevTools or some of state management library DevTools? |
Looks great! Do you have an idea on how will it play with |
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
@huntie @cortinico As a library maintainer, the Realm team has a few comments:
|
JSC debugging, on-device debugging
On the future of Flipper and custom plugins
|
@huntie Thank you for the comments, and addressing our concerns. We are happy to see that you also believe that debugging on device is here to stay (and we don't need to maintain a poor shadow implementation of Realm). Of course it will be ideal to support both JSC and Hermes, but for now it is fine to push users in the direction of Hermes. |
a1a6d3a
to
6aa29a8
Compare
Update: Amended timelines Just pushed an edit following some of this feedback and our internal discussions.
Note: React Native release dates are not official — just my best guess. |
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.
LGTM
@huntie could you please also change the PR desc to add ⬆️. Not much but the PR link previews are somewhat misleading ;) |
BREAKING CHANGE: Dropped support for Flipper. For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper. For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper. For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper. For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper. For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper. For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper. For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper. For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper For more context, see react-native-community/discussions-and-proposals#641.
is there a proposal for a network tab ? being able to look at network requests is in many cases critical for troubleshooting |
@jbrodriguez Debugging via the Network tab in Chrome DevTools is definitely in our long term plans! We are likely to begin working on this in 2024 (no promises on when this ships, though!). We will be prefacing this work with some necessary foundational improvements to how we implement CDP in React Native. We will be sharing more of our debugging plans publicly in future. Until then, here's a couple of links:
|
BREAKING CHANGE: Dropped support for Flipper For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper For more context, see react-native-community/discussions-and-proposals#641.
BREAKING CHANGE: Dropped support for Flipper For more context, see react-native-community/discussions-and-proposals#641.
Well done, RN team! Drop Flipper support and providing nothing for Network debugging instead. You've made my life easier. |
Outlines our intent to no longer include Flipper's native integration in React Native 0.74, and to replace Flipper's JS debugging experience in React Native 0.73 with an equivalent one-click debugging flow leveraging Chrome DevTools.
View the rendered RFC