Skip to content

Conversation

md5
Copy link

@md5 md5 commented Aug 18, 2025

I noticed that there were a number of broken links to documentation for AudioParam in the p5.sound docs, pointing to the defunct wiki on webplatform.org. This PR updates to point to the AudioParam docs in the Web Audio spec instead.

There was an additional reference to the broken URL in public/reference/data.json, but it seems like that might be a generated file. I can update that reference as well if that's necessary or desirable.

Copy link
Contributor

@perminder-17 perminder-17 left a comment

Choose a reason for hiding this comment

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

Hi @md5, thanks a lot for catching all these broken links and for fixing them. The only issue is that the p5.Sound docs are auto-generated from the p5.sound repo. So the best approach would be to make the changes directly in that repo instead. What you think?

@md5
Copy link
Author

md5 commented Aug 22, 2025

Closed in favor of processing/p5.js-sound#808

@md5 md5 closed this Aug 22, 2025
@md5 md5 deleted the fix/p5-sound-audio-param-links branch August 22, 2025 06:31
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.

2 participants