Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • C create-react-app
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1,547
    • Issues 1,547
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 417
    • Merge requests 417
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Meta
  • create-react-app
  • Issues
  • #10539
Closed
Open
Issue created Feb 10, 2021 by Administrator@rootContributor

FAST_REFRESH behaviour

Created by: hyperknot

  1. FAST_REFRESH is mentioned as experimental / unstable in the original release log here: https://github.com/facebook/create-react-app/pull/8582 and here: https://github.com/facebook/create-react-app/blob/3f5dea91c1721b233e5a2b514ff9ea6fbc76ff97/packages/react-scripts/config/env.js#L97-L98

  2. 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.

  3. Against what is written in the release log and source code comments, it is actually ENABLED by default today. https://github.com/facebook/create-react-app/blob/3f5dea91c1721b233e5a2b514ff9ea6fbc76ff97/packages/react-scripts/config/env.js#L100

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.

Assignee
Assign to
Time tracking