-
-
Notifications
You must be signed in to change notification settings - Fork 26.8k
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
FAST_REFRESH behaviour #10539
Comments
I noticed an issue with the fast refresh and Apollo local state today as well. If I have fast refresh enabled, the very first value I get from the cache is |
Initially, I thought Now, live edits ALWAYS FAIL TO REFRESH. While I have high hopes for Fast Refresh, the OPT-OUT workaround is a must:
|
FAST_REFRESH is mentioned as experimental / unstable in the original release log
here: Add experimental react-refresh support #8582 and here:
create-react-app/packages/react-scripts/config/env.js
Lines 97 to 98 in 3f5dea9
I can confirm it's very unstable / unusable today. It doesn't detect any change in non-UI components, for example MobX stores, thus the whole refresh is broken, I always have to reload manually now.
Against what is written in the release log and source code comments, it is actually ENABLED by default today.
create-react-app/packages/react-scripts/config/env.js
Line 100 in 3f5dea9
I believe this line need to be changed to
=== 'true'
, to make it disabled by default. Also, a small explanation should be added to the next release log, this feature is really not documented at all.The text was updated successfully, but these errors were encountered: