Javascript / Factory functions: Move the object shorthand notation to Foundations / Object basics #29442
Replies: 3 comments
-
Thank you for taking the time to make this suggestion. I'd like someone on the @TheOdinProject/javascript team to review this one to help determine the next step. |
Beta Was this translation helpful? Give feedback.
-
Converting to a discussion until we have something actionable |
Beta Was this translation helpful? Give feedback.
-
#29521 will address the second point regarding content on arrays in the Object Basics lesson. After review and it gets merged, it should be easier for the author to make a more concrete proposal about their main suggestion, as the Object Basics lesson will be more focused on, well, object basics :) |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Checks
Describe your suggestion
The Factory Functions lesson is pretty packed with info and I feel that the Object Shorthand Notation note is quite out of place, inside the Factory Functions heading. It even links to the MDN documentation on the destructuring assigment, which I feel is quite a detour from the subject at hand.
I propose moving that note to Object basics lesson on the foundations path. In that lesson, the content about objects is almost minimal, so I feel there would be the most appropriate place.
In a related manner, that Object basics lesson includes two very different things: objects, and array functions. And the assignments are actually all about the array functions, even that the name of the lesson is
Object basics
. Now, having this Object Shorthand Notation inside the foundations lesson would allow for that lesson to be longer and therefore split in two,Object basics
andArray methods
Path
Foundations
Node / JS
Lesson Url
https://www.theodinproject.com/lessons/node-path-javascript-factory-functions-and-the-module-pattern#factory-functions
(Optional) Discord Name
No response
(Optional) Additional Comments
No response
Beta Was this translation helpful? Give feedback.
All reactions