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

Classes: Contradictory info #28797

Open
2 of 3 tasks
hi-its-jack opened this issue Sep 16, 2024 · 3 comments · May be fixed by #28947
Open
2 of 3 tasks

Classes: Contradictory info #28797

hi-its-jack opened this issue Sep 16, 2024 · 3 comments · May be fixed by #28947
Assignees

Comments

@hi-its-jack
Copy link

Checks

Describe your suggestion

Hello, in the overview you state that there isn't much new to learn and then this is followed with quite a lot of learning objectives and reading material. Seems a bit of contradiction. Maybe needs changing?

Path

Node / JS

Lesson Url

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

(Optional) Discord Name

No response

(Optional) Additional Comments

No response

@MaoShizhong
Copy link
Contributor

Thanks for the issue @hi-its-jack. I've personally had some ideas already about the JS Classes lesson, and I agree with you that the wording is a bit strange there. It does make sense, classes aren't really a new concept/mechanic in JS - they're almost entirely syntactic sugar for constructor functions and prototypes people would have already learnt about. So there's no big new mechanics to learn like with the factory functions lesson, for example.

That being said, the assignment section has gotten a bit out of hand IMHO, so I can totally understand someone coming in new going "huh? That's not a little..." even though a lot of it is just alternative (and nicer) syntax for some things.

I'll bring this up with the team, as I've set out some plans for a minor rewrite of that lesson that'll hopefully make the above clearer.

@hi-its-jack
Copy link
Author

hi-its-jack commented Sep 17, 2024 via email

@MaoShizhong MaoShizhong self-assigned this Oct 5, 2024
@S-yartey007
Copy link

An amazing resource indeed.

@MaoShizhong MaoShizhong linked a pull request Oct 12, 2024 that will close this issue
7 tasks
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

Successfully merging a pull request may close this issue.

3 participants