Skip to content

docs: update remaining loadSound examples to use async/await - #109

Open
NisargJasani0602 wants to merge 1 commit into
processing:mainfrom
NisargJasani0602:docs/loadsound-preload-examples
Open

docs: update remaining loadSound examples to use async/await#109
NisargJasani0602 wants to merge 1 commit into
processing:mainfrom
NisargJasani0602:docs/loadsound-preload-examples

Conversation

@NisargJasani0602

Copy link
Copy Markdown
Contributor

Summary

  • Panner.js, Panner3D.js, Amplitude.js, and Gain.js still documented loadSound() being called from preload(), which is stale now that loadSound() uses an async/await pattern (see 86c3bcf).
  • Updated all four to use async function setup() + await loadSound(...), consistent with SoundFile.js, Reverb.js, and Delay.js.

Refs #106

Test plan

  • Verify JSDoc examples render correctly on the docs site
  • Confirm no other files reference loadSound() inside preload()

Panner, Panner3D, Amplitude, and Gain still documented loadSound()
being called from preload(), which is stale now that loadSound()
uses an async/await pattern (see 86c3bcf).

Refs processing#106

@ksen0 ksen0 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice, thank you! @ogbabydiesal can you check please if this is alright and review / merge as you see fit? Thanks both!!

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