Skip to content

Point all Webpack links to 1.x documentation#8697

Merged
aweary merged 2 commits intofacebook:masterfrom
aweary:fix-webpack-doc-links
Jan 6, 2017
Merged

Point all Webpack links to 1.x documentation#8697
aweary merged 2 commits intofacebook:masterfrom
aweary:fix-webpack-doc-links

Conversation

@aweary
Copy link
Copy Markdown
Contributor

@aweary aweary commented Jan 5, 2017

Comment thread docs/docs/installation.md Outdated
#### Webpack

Include both `DefinePlugin` and `UglifyJsPlugin` into your production Webpack configuration as described in [this guide](https://webpack.js.org/guides/production-build/).
Include both [`DefinePlugin`](https://webpack.github.io/docs/list-of-plugins.html#defineplugin) and [`UglifyJsPlugin`](https://webpack.github.io/docs/list-of-plugins.html#uglifyjsplugin) into your production Webpack configuration.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I couldn't find an equivalent guide on production builds on the 1.x site, so I just linked to the docs for each plugin instead.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this particular case I'd rather link to 2.x doc because it contains specific actionable advice, and works identically in 1.x.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My only worry with that is users clicking to the other linked doc pages from that tutorial and not really realizing they're potentially getting bad information.

We could mention that this article is on the Webpack 2 doc site after linking to it? Something like:

This guide works with Webpack 1.x and 2, but is hosted on the new Webpack 2 site. If you're not using the Webpack 2 beta, refer to the Webpack 1.x website for all other documentation.

Otherwise, I can just revert it back 👍

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sentence looks fine to me.

Copy link
Copy Markdown
Collaborator

@gaearon gaearon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sheep it

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.

3 participants