8000 Javascript / Factory functions: Move the object shorthand notation to Foundations / Object basics · Issue #29413 · TheOdinProject/curriculum · GitHub
[go: up one dir, main page]

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Javascript / Factory functions: Move the object shorthand notation to Foundations / Object basics #29413

Open
2 of 3 tasks
Asmatzaile opened this issue Feb 14, 2025 · 1 comment

Comments

@Asmatzaile
Copy link
Contributor
Asmatzaile commented Feb 14, 2025

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 and Array 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

@rlmoser99
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants
0