p5.js-website icon indicating copy to clipboard operation
p5.js-website copied to clipboard

Update “Getting Started with Node.js” tutorial for p5.js 2.x

Open perminder-17 opened this issue 7 months ago • 4 comments

Most appropriate sections of the p5.js website?

Tutorials

What is your operating system?

Linux

Web browser and version

Firefox

Actual Behavior

The “Getting Started with Node.js” tutorial is still using p5.js 1.x patterns (e.g. preload(), old APIs, external sketches locked to 1.x). Legacy p5.js 1.x APIs (e.g. old shape-drawing signatures) remain in the code samples. Several external Web Editor sketches are locked to p5.js 1.x and won’t run under 2.x.

Expected Behavior

Since p5.js 2.0 has removed preload() and other legacy methods in favor of async/await and updated several APIs, we need to refresh this tutorial so it only uses supported 2.x functionality. Every linked sketch should has been forked (or updated) to use p5.js 2.x so examples execute correctly.

Steps to reproduce

No response

Would you like to work on the issue?

Looking for volunteer

perminder-17 avatar May 10 '25 16:05 perminder-17

Thanks for spotting this! In this one, I believe the only external code examples is https://github.com/MsQCompSci/melodyAppSetup so this could potentially be forked for a 2.0 version? Is there other external code in this?

ksen0 avatar May 11 '25 12:05 ksen0

@perminder-17 I saw the outdated code blocks which need to be updated to match v2.x syntax. Not sure which would be faster, forking the Melody app repo or submitting a pull request on the original repository ?

@ksen0 I'd love to work on this at the earliest convenience.

prjctimg avatar May 12 '25 13:05 prjctimg

hi, made a pr, checkout here: https://github.com/processing/p5.js-website/pull/877 @perminder-17

NalinDalal avatar Jun 26 '25 06:06 NalinDalal

Hi @prjctimg so sorry I wasn't able to assign this to you? Are you still interested in working on it?

Btw @NalinDalal has already opend up a PR? is it fine if you review his work and request changes?

perminder-17 avatar Jun 26 '25 09:06 perminder-17

Attached PR has been merged, thanks all!

ksen0 avatar Jul 18 '25 16:07 ksen0