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.

Sign up for GitHub

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

Jump to bottom

Update: NodeJS_Docker_Cheat-Sheet #1620

Closed
jtcmedia opened this issue Mar 2, 2025 · 3 comments · Fixed by #1624
Closed

Update: NodeJS_Docker_Cheat-Sheet #1620

jtcmedia opened this issue Mar 2, 2025 · 3 comments · Fixed by #1624
Assignees
@deep231w
Labels
ACK_OBTAINED Issue acknowledged from core team so work can be done to fix it. UPDATE_CS Issue about the update/refactoring of a existing cheat sheet.

Comments

@jtcmedia
Copy link

jtcmedia commented Mar 2, 2025

What is missing or needs to be updated?

In the section 3) Optimize Node.js tooling for production, there is a dead link (403 Error) to a screenshot (Express documentation screenshot) .

How should this be resolved?

Remove the link. It isn't really necessary.
@jtcmedia jtcmedia added ACK_WAITING Issue waiting acknowledgement from core team before to start the work to fix it. HELP_WANTED Issue for which help is wanted to do the job. UPDATE_CS Issue about the update/refactoring of a existing cheat sheet. labels Mar 2, 2025
@szh
Copy link
Collaborator

szh commented Mar 3, 2025

Makes sense, do you want to submit a PR for this?

@szh szh added ACK_OBTAINED Issue acknowledged from core team so work can be done to fix it. and removed ACK_WAITING Issue waiting acknowledgement from core team before to start the work to fix it. labels Mar 3, 2025
@deep231w
Copy link
Contributor

deep231w commented Mar 8, 2025

I'd like to work on this. Can you assign me?

@mackowski
Copy link
Collaborator

sure @deep231w

@mackowski mackowski removed the HELP_WANTED Issue for which help is wanted to do the job. label Mar 10, 2025
@szh szh linked a pull request Mar 10, 2025 that will close this issue
Removed dead link in NodeJS Docker Cheat Sheet #1624
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@deep231w deep231w

Labels
ACK_OBTAINED Issue acknowledged from core team so work can be done to fix it. UPDATE_CS Issue about the update/refactoring of a existing cheat sheet.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Removed dead link in NodeJS Docker Cheat Sheet deep231w/CheatSheetSeries
4 participants
@szh @jtcmedia @mackowski @deep231w