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

Getting Started NodeJS Course - New 2024 TraversyMedia crash course video suggestion. #27865

Open
2 of 3 tasks
h4m24-a opened this issue Apr 26, 2024 · 7 comments
Open
2 of 3 tasks

Comments

@h4m24-a
Copy link

h4m24-a commented Apr 26, 2024

Checks

Describe your suggestion

In the additional resources the linked TraversyMedia crash course was uploaded 5 years ago and a new 2024 Node Crash course was uploaded a few days ago by TraversyMedia.

I know this may not be necessary so its just a suggestion.

You could add both videos.

https://www.youtube.com/watch?v=32M1al-Y6Ag

Path

Node / JS

Lesson Url

https://www.theodinproject.com/lessons/nodejs-getting-started#additional-resources

(Optional) Discord Name

No response

(Optional) Additional Comments

No response

@Siriuszx
Copy link
Contributor

I would like to be assigned to this issue 🙂

@wise-king-sullyman
Copy link
Member

Thanks for creating this issue!

@Siriuszx thank you for volunteering! I'm happy to assign you, but I don't think we would want to have both, and we'd probably need to make the call on which we'd rather have. The older one being older doesn't necessarily mean that it doesn't fit our use case better.

The scope here would really have to be investigating the new video and watching the old video if you haven't already, documenting the differences (particularly what, if anything, was in the old one but is not in the new one), and providing that information to the maintainers.

Before you do anything though, @TheOdinProject/javascript what do we think about the additional resources on this lesson in general? Have any of you already watched it all? It's quite lengthy, the first video is over an hour and a half and the second resource is a 12 part series. I wonder if we want to include them in the first place.

@Siriuszx
Copy link
Contributor

Great! If you need help with this I would like to volunteer portion of my time for new content review etc.

@MaoShizhong
Copy link
Contributor

@wise-king-sullyman Just watched the old and new videos over last night and today.
I don't think the new version offers anything significant that warrants a replacement for the old version, i.e. the old version is still perfectly suitable for our needs and the context of our current curriculum. In fact, it's kinda slightly more suitable given the old one uses CJS, and the new one uses ESM. Our current curriculum's resources all use CJS by default as far as I'm aware.

So I don't think any replacement needs to be done in this case.

Since we are revamping the Node course, this may well be a source to consult when we get to the Node intro section (we're still working on the Express section). Since by that point, we'll have a lot more control over what material we write, we may not even include the new course as an AR link but use it to write our own in-house material which we have more control over maintenance for.

@wise-king-sullyman
Copy link
Member

@Siriuszx thanks for the offer! 💪

@MaoShizhong thanks for digging into it. Having watched it do you think it's something that is worth keeping as an AR at all for now?

@MaoShizhong
Copy link
Contributor

@wise-king-sullyman I think we should just keep the current AR as it is, and make no changes.

@wise-king-sullyman
Copy link
Member

I'll leave this open for a bit in case other maintainers would like to add any thoughts, but if nothing else happens it seems we should probably close this issue after a week or two.

Thanks again though @h4m24-a and @Siriuszx !

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

4 participants