Skip to content

Update links for code splitting and dependencies#11852

Open
dtran320 wants to merge 1 commit into
react:mainfrom
dtran320:patch-1
Open

Update links for code splitting and dependencies#11852
dtran320 wants to merge 1 commit into
react:mainfrom
dtran320:patch-1

Conversation

@dtran320
Copy link
Copy Markdown

Save users a few redirects and 2 clicks by linking to the current docs for code splitting and analyzing bundle size.

More specifically, previously, the link for code splitting pointed to https://goo.gl/9VhYWB, which pointed to https://github.com/facebookincubator/create-react-app/blob/master/packages/react-scripts/template/README.md#code-splitting, which redirects to https://github.com/facebook/create-react-app/blob/main/packages/react-scripts/template/README.md#code-splitting, which displayed a "This file has moved here" pointing to https://github.com/facebook/create-react-app/blob/main/packages/cra-template/template/README.md, which links to the new docs: https://create-react-app.dev/docs/code-splitting/

Similarly, previously, the link for analyzing dependencies/bundle size pointed to https://goo.gl/LeUzfb, which pointed to https://github.com/facebookincubator/create-react-app/blob/master/packages/react-scripts/template/README.md#analyzing-the-bundle-size, which redirected to https://github.com/facebook/create-react-app/blob/main/packages/react-scripts/template/README.md#analyzing-the-bundle-size, which displayed a message "This file has moved here" pointing to https://github.com/facebook/create-react-app/blob/main/packages/cra-template/template/README.md, which links to the new docs: https://create-react-app.dev/docs/analyzing-the-bundle-size/

The bundle size is significantly larger than recommended.
Consider reducing it with code splitting: https://goo.gl/9VhYWB
You can also analyze the project dependencies: https://goo.gl/LeUzfb

file_has_moved

file_has_moved_2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants