diff --git a/app/react/src/server/cra_config.js b/app/react/src/server/cra_config.js index 4a8cdda3339e..23d3a781d2d0 100644 --- a/app/react/src/server/cra_config.js +++ b/app/react/src/server/cra_config.js @@ -64,14 +64,7 @@ export function applyCRAWebpackConfig(baseConfig) { // Add css minification for production const plugins = [...baseConfig.plugins]; if (baseConfig.mode === 'production') { - plugins.push( - new MiniCssExtractPlugin({ - // Options similar to the same options in webpackOptions.output - // both options are optional - filename: 'static/css/[name].[contenthash:8].css', - chunkFilename: 'static/css/[name].[contenthash:8].chunk.css', - }) - ); + plugins.push(new MiniCssExtractPlugin()); } return { diff --git a/lib/cli/test/fixtures/react_scripts_v2/.gitignore b/lib/cli/test/fixtures/react_scripts_v2/.gitignore new file mode 100644 index 000000000000..4d29575de804 --- /dev/null +++ b/lib/cli/test/fixtures/react_scripts_v2/.gitignore @@ -0,0 +1,23 @@ +# See https://help.github.com/articles/ignoring-files/ for more about ignoring files. + +# dependencies +/node_modules +/.pnp +.pnp.js + +# testing +/coverage + +# production +/build + +# misc +.DS_Store +.env.local +.env.development.local +.env.test.local +.env.production.local + +npm-debug.log* +yarn-debug.log* +yarn-error.log* diff --git a/lib/cli/test/fixtures/react_scripts_v2/README.md b/lib/cli/test/fixtures/react_scripts_v2/README.md new file mode 100644 index 000000000000..897dc8366019 --- /dev/null +++ b/lib/cli/test/fixtures/react_scripts_v2/README.md @@ -0,0 +1,44 @@ +This project was bootstrapped with [Create React App](https://github.com/facebook/create-react-app). + +## Available Scripts + +In the project directory, you can run: + +### `npm start` + +Runs the app in the development mode.
+Open [http://localhost:3000](http://localhost:3000) to view it in the browser. + +The page will reload if you make edits.
+You will also see any lint errors in the console. + +### `npm test` + +Launches the test runner in the interactive watch mode.
+See the section about [running tests](https://facebook.github.io/create-react-app/docs/running-tests) for more information. + +### `npm run build` + +Builds the app for production to the `build` folder.
+It correctly bundles React in production mode and optimizes the build for the best performance. + +The build is minified and the filenames include the hashes.
+Your app is ready to be deployed! + +See the section about [deployment](https://facebook.github.io/create-react-app/docs/deployment) for more information. + +### `npm run eject` + +**Note: this is a one-way operation. Once you `eject`, you can’t go back!** + +If you aren’t satisfied with the build tool and configuration choices, you can `eject` at any time. This command will remove the single build dependency from your project. + +Instead, it will copy all the configuration files and the transitive dependencies (Webpack, Babel, ESLint, etc) right into your project so you have full control over them. All of the commands except `eject` will still work, but they will point to the copied scripts so you can tweak them. At this point you’re on your own. + +You don’t have to ever use `eject`. The curated feature set is suitable for small and middle deployments, and you shouldn’t feel obligated to use this feature. However we understand that this tool wouldn’t be useful if you couldn’t customize it when you are ready for it. + +## Learn More + +You can learn more in the [Create React App documentation](https://facebook.github.io/create-react-app/docs/getting-started). + +To learn React, check out the [React documentation](https://reactjs.org/). diff --git a/lib/cli/test/fixtures/react_scripts_v2/package.json b/lib/cli/test/fixtures/react_scripts_v2/package.json new file mode 100644 index 000000000000..c1d8c6333a70 --- /dev/null +++ b/lib/cli/test/fixtures/react_scripts_v2/package.json @@ -0,0 +1,22 @@ +{ + "name": "react-scripts-v2-fixture", + "version": "0.1.0", + "private": true, + "dependencies": { + "react": "^16.6.0", + "react-dom": "^16.6.0", + "react-scripts": "2.1.1" + }, + "scripts": { + "start": "react-scripts start", + "build": "react-scripts build", + "test": "react-scripts test", + "eject": "react-scripts eject" + }, + "browserslist": [ + ">0.2%", + "not dead", + "not ie <= 11", + "not op_mini all" + ] +} diff --git a/lib/cli/test/fixtures/react_scripts_v2/public/favicon.ico b/lib/cli/test/fixtures/react_scripts_v2/public/favicon.ico new file mode 100644 index 000000000000..a11777cc471a Binary files /dev/null and b/lib/cli/test/fixtures/react_scripts_v2/public/favicon.ico differ diff --git a/lib/cli/test/fixtures/react_scripts_v2/public/index.html b/lib/cli/test/fixtures/react_scripts_v2/public/index.html new file mode 100644 index 000000000000..4bfce95684f8 --- /dev/null +++ b/lib/cli/test/fixtures/react_scripts_v2/public/index.html @@ -0,0 +1,40 @@ + + + + + + + + + + + React App + + + +
+ + + diff --git a/lib/cli/test/fixtures/react_scripts_v2/public/manifest.json b/lib/cli/test/fixtures/react_scripts_v2/public/manifest.json new file mode 100644 index 000000000000..1f2f141fafde --- /dev/null +++ b/lib/cli/test/fixtures/react_scripts_v2/public/manifest.json @@ -0,0 +1,15 @@ +{ + "short_name": "React App", + "name": "Create React App Sample", + "icons": [ + { + "src": "favicon.ico", + "sizes": "64x64 32x32 24x24 16x16", + "type": "image/x-icon" + } + ], + "start_url": ".", + "display": "standalone", + "theme_color": "#000000", + "background_color": "#ffffff" +} diff --git a/lib/cli/test/fixtures/react_scripts_v2/src/App.css b/lib/cli/test/fixtures/react_scripts_v2/src/App.css new file mode 100644 index 000000000000..92f956e80402 --- /dev/null +++ b/lib/cli/test/fixtures/react_scripts_v2/src/App.css @@ -0,0 +1,32 @@ +.App { + text-align: center; +} + +.App-logo { + animation: App-logo-spin infinite 20s linear; + height: 40vmin; +} + +.App-header { + background-color: #282c34; + min-height: 100vh; + display: flex; + flex-direction: column; + align-items: center; + justify-content: center; + font-size: calc(10px + 2vmin); + color: white; +} + +.App-link { + color: #61dafb; +} + +@keyframes App-logo-spin { + from { + transform: rotate(0deg); + } + to { + transform: rotate(360deg); + } +} diff --git a/lib/cli/test/fixtures/react_scripts_v2/src/App.js b/lib/cli/test/fixtures/react_scripts_v2/src/App.js new file mode 100644 index 000000000000..7e261ca47e6b --- /dev/null +++ b/lib/cli/test/fixtures/react_scripts_v2/src/App.js @@ -0,0 +1,28 @@ +import React, { Component } from 'react'; +import logo from './logo.svg'; +import './App.css'; + +class App extends Component { + render() { + return ( +
+
+ logo +

+ Edit src/App.js and save to reload. +

+ + Learn React + +
+
+ ); + } +} + +export default App; diff --git a/lib/cli/test/fixtures/react_scripts_v2/src/App.test.js b/lib/cli/test/fixtures/react_scripts_v2/src/App.test.js new file mode 100644 index 000000000000..a754b201bf9c --- /dev/null +++ b/lib/cli/test/fixtures/react_scripts_v2/src/App.test.js @@ -0,0 +1,9 @@ +import React from 'react'; +import ReactDOM from 'react-dom'; +import App from './App'; + +it('renders without crashing', () => { + const div = document.createElement('div'); + ReactDOM.render(, div); + ReactDOM.unmountComponentAtNode(div); +}); diff --git a/lib/cli/test/fixtures/react_scripts_v2/src/index.css b/lib/cli/test/fixtures/react_scripts_v2/src/index.css new file mode 100644 index 000000000000..cee5f348fb90 --- /dev/null +++ b/lib/cli/test/fixtures/react_scripts_v2/src/index.css @@ -0,0 +1,14 @@ +body { + margin: 0; + padding: 0; + font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", "Roboto", "Oxygen", + "Ubuntu", "Cantarell", "Fira Sans", "Droid Sans", "Helvetica Neue", + sans-serif; + -webkit-font-smoothing: antialiased; + -moz-osx-font-smoothing: grayscale; +} + +code { + font-family: source-code-pro, Menlo, Monaco, Consolas, "Courier New", + monospace; +} diff --git a/lib/cli/test/fixtures/react_scripts_v2/src/index.js b/lib/cli/test/fixtures/react_scripts_v2/src/index.js new file mode 100644 index 000000000000..0c5e75da1cd0 --- /dev/null +++ b/lib/cli/test/fixtures/react_scripts_v2/src/index.js @@ -0,0 +1,12 @@ +import React from 'react'; +import ReactDOM from 'react-dom'; +import './index.css'; +import App from './App'; +import * as serviceWorker from './serviceWorker'; + +ReactDOM.render(, document.getElementById('root')); + +// If you want your app to work offline and load faster, you can change +// unregister() to register() below. Note this comes with some pitfalls. +// Learn more about service workers: http://bit.ly/CRA-PWA +serviceWorker.unregister(); diff --git a/lib/cli/test/fixtures/react_scripts_v2/src/logo.svg b/lib/cli/test/fixtures/react_scripts_v2/src/logo.svg new file mode 100644 index 000000000000..6b60c1042f58 --- /dev/null +++ b/lib/cli/test/fixtures/react_scripts_v2/src/logo.svg @@ -0,0 +1,7 @@ + + + + + + + diff --git a/lib/cli/test/fixtures/react_scripts_v2/src/serviceWorker.js b/lib/cli/test/fixtures/react_scripts_v2/src/serviceWorker.js new file mode 100644 index 000000000000..2283ff9ced12 --- /dev/null +++ b/lib/cli/test/fixtures/react_scripts_v2/src/serviceWorker.js @@ -0,0 +1,135 @@ +// This optional code is used to register a service worker. +// register() is not called by default. + +// This lets the app load faster on subsequent visits in production, and gives +// it offline capabilities. However, it also means that developers (and users) +// will only see deployed updates on subsequent visits to a page, after all the +// existing tabs open on the page have been closed, since previously cached +// resources are updated in the background. + +// To learn more about the benefits of this model and instructions on how to +// opt-in, read http://bit.ly/CRA-PWA + +const isLocalhost = Boolean( + window.location.hostname === 'localhost' || + // [::1] is the IPv6 localhost address. + window.location.hostname === '[::1]' || + // 127.0.0.1/8 is considered localhost for IPv4. + window.location.hostname.match( + /^127(?:\.(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)){3}$/ + ) +); + +export function register(config) { + if (process.env.NODE_ENV === 'production' && 'serviceWorker' in navigator) { + // The URL constructor is available in all browsers that support SW. + const publicUrl = new URL(process.env.PUBLIC_URL, window.location.href); + if (publicUrl.origin !== window.location.origin) { + // Our service worker won't work if PUBLIC_URL is on a different origin + // from what our page is served on. This might happen if a CDN is used to + // serve assets; see https://github.com/facebook/create-react-app/issues/2374 + return; + } + + window.addEventListener('load', () => { + const swUrl = `${process.env.PUBLIC_URL}/service-worker.js`; + + if (isLocalhost) { + // This is running on localhost. Let's check if a service worker still exists or not. + checkValidServiceWorker(swUrl, config); + + // Add some additional logging to localhost, pointing developers to the + // service worker/PWA documentation. + navigator.serviceWorker.ready.then(() => { + console.log( + 'This web app is being served cache-first by a service ' + + 'worker. To learn more, visit http://bit.ly/CRA-PWA' + ); + }); + } else { + // Is not localhost. Just register service worker + registerValidSW(swUrl, config); + } + }); + } +} + +function registerValidSW(swUrl, config) { + navigator.serviceWorker + .register(swUrl) + .then(registration => { + registration.onupdatefound = () => { + const installingWorker = registration.installing; + if (installingWorker == null) { + return; + } + installingWorker.onstatechange = () => { + if (installingWorker.state === 'installed') { + if (navigator.serviceWorker.controller) { + // At this point, the updated precached content has been fetched, + // but the previous service worker will still serve the older + // content until all client tabs are closed. + console.log( + 'New content is available and will be used when all ' + + 'tabs for this page are closed. See http://bit.ly/CRA-PWA.' + ); + + // Execute callback + if (config && config.onUpdate) { + config.onUpdate(registration); + } + } else { + // At this point, everything has been precached. + // It's the perfect time to display a + // "Content is cached for offline use." message. + console.log('Content is cached for offline use.'); + + // Execute callback + if (config && config.onSuccess) { + config.onSuccess(registration); + } + } + } + }; + }; + }) + .catch(error => { + console.error('Error during service worker registration:', error); + }); +} + +function checkValidServiceWorker(swUrl, config) { + // Check if the service worker can be found. If it can't reload the page. + fetch(swUrl) + .then(response => { + // Ensure service worker exists, and that we really are getting a JS file. + const contentType = response.headers.get('content-type'); + if ( + response.status === 404 || + (contentType != null && contentType.indexOf('javascript') === -1) + ) { + // No service worker found. Probably a different app. Reload the page. + navigator.serviceWorker.ready.then(registration => { + registration.unregister().then(() => { + window.location.reload(); + }); + }); + } else { + // Service worker found. Proceed as normal. + registerValidSW(swUrl, config); + } + }) + .catch(() => { + console.log( + 'No internet connection found. App is running in offline mode.' + ); + }); +} + +export function unregister() { + if ('serviceWorker' in navigator) { + navigator.serviceWorker.ready.then(registration => { + registration.unregister(); + }); + } +}