Skip to content

Sound examples - #1232

Merged
ksen0 merged 16 commits into
processing:mainfrom
ogbabydiesal:sound-examples
Nov 8, 2026
Merged

ksen0 merged 16 commits into
processing:mainfrom
ogbabydiesal:sound-examples

Conversation

@ogbabydiesal

@ogbabydiesal ogbabydiesal commented Mar 12, 2026 •

Copy link
Copy Markdown
Member

@ogbabydiesal
ogbabydiesal changed the base branch from main to 2.0 March 12, 2026 12:58
@Nwakaego-Ego

Copy link
Copy Markdown
Contributor

Hi @ogbabydiesal, I know this PR is still a draft. Before marking it ready to review, here are a few things to address:

Please add a description to the PR, also link the related issue using "Resolves #issue number" in the description
The featured image in description.mdx is currently pointing to the Async/Await thumbnail. Please confirm if this is a placeholder or the correct image for this example. The commit message appears to be cut off; update it to be more descriptive. Thank you.

@ksen0

ksen0 commented May 18, 2026

Copy link
Copy Markdown
Member

Hi @Nwakaego-Ego thanks for the notes! @ogbabydiesal and I are working on this one together, there's a couple things left to do in addition to the points you mentioned

Comment thread src/globals/p5-version.ts Resolved
@ogbabydiesal

Copy link
Copy Markdown
Member Author

hi @Nwakaego-Ego and @ksen0 this pr is for sound examples and simple-melody-tutorial

@ksen0
ksen0 marked this pull request as ready for review September 3, 2026 10:17
@ksen0

ksen0 commented Sep 3, 2026

Copy link
Copy Markdown
Member

I've marked it as ready for review based on Discord chat! I will take a look this week, thanks so much @ogbabydiesal

@ksen0

ksen0 commented Nov 8, 2026

Copy link
Copy Markdown
Member

Update: I am working on getting this merged asap with the new sound version. Sorry for the delays!

@ksen0
ksen0 merged commit 8c5e2c7 into processing:main Nov 8, 2026
4 checks passed
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 this pull request may close these issues.

3 participants