curriculum icon indicating copy to clipboard operation
curriculum copied to clipboard

Webpack: Move webpack merge/modes mention to a later lesson

Open MaoShizhong opened this issue 6 months ago • 5 comments

Checks

Describe your suggestion

In #26381 and #26552 I ended up adding step 4 of the webpack lesson assignment which took the user through the difference between development/production mode and using webpack-merge to set a dev and prod config tied to their own npm scripts.

Whilst this has proved convenient for some people (especially after more experience with Webpack) and has reduced the number of questions concerning the warning when no mode is specified, in hindsight, I feel it is not necessary to include that part in the Webpack lesson as it's not strictly necessary to get started with it. I feel it would be more beneficial moved to a later lesson e.g. the linting lesson (though still not 100% sure if this would the best place either?) in order to give the learner more time to use and get a feel for webpack before then introducing convenience measures.

This came about starting with this Discord discussion and keeping an eye on things since then, whilst fewer Qs about webpack mode came up, there seemed to be more confusing about webpack as a whole than needs be as people were already struggling to get to grips with the more important parts of using webpack let alone dealing with the difference between mode configs (if anything, the less unnecessary config they need to do at the start, the simpler the learning curve would be I feel). Html-webpack-plugin and css/style loaders I feel are the primary ones needed for straightforward workflow and understanding the roles of src/dist and should be plenty to get through restaurant page and to-do list. It just feels currently that webpack-merge and modes will serve only to overwhelm at the point they're mentioned.

Thoughts?

Path

Ruby / Rails, Node / JS

Lesson Url

https://www.theodinproject.com/lessons/node-path-javascript-webpack

(Optional) Discord Name

Mao

(Optional) Additional Comments

No response

MaoShizhong avatar Dec 23 '23 10:12 MaoShizhong