Skip to content

Conversation

ridwanyinus
Copy link
Contributor

Because

The NodeJS Crash Course URL points to an outdated version.

This PR

Issue

Closes #XXXXX

Additional Information

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project curriculum contributing guide
  • The title of this PR follows the location of change: brief description of change format, e.g. Intro to HTML and CSS lesson: Fix link text
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If any lesson files are included in this PR, they have been previewed with the Markdown preview tool to ensure it is formatted correctly
  • If any lesson files are included in this PR, they follow the Layout Style Guide

@github-actions github-actions bot added the Content: NodeJS Involves the NodeJS course label Sep 18, 2025
@JoshDevHub
Copy link
Contributor

JoshDevHub commented Sep 21, 2025

Thank you for making this PR @ridwanyinus. I'm going to assign a member of our @TheOdinProject/javascript team to get their input on this.

@JoshDevHub JoshDevHub requested review from a team and bycdiaz and removed request for a team September 21, 2025 15:06
Copy link
Member

@bycdiaz bycdiaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating this!

@ridwanyinus
Copy link
Contributor Author

Thanks for updating this!

my pleasure sir

@mao-sz
Copy link
Contributor

mao-sz commented Oct 2, 2025

@bycdiaz saw this hasn't been merged yet. Thought I'd let you know about #27865 for why this change has previously been rejected.

That being said, on another look, given our changing and stricter outlook on additional resources, would you agree that it may be better to just remove this resource? The previous one is still more in line with the current Node course (revamp still WIP with the earlier lessons to be addressed in a future milestone).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content: NodeJS Involves the NodeJS course
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants