-
-
Notifications
You must be signed in to change notification settings - Fork 129
[Snippets]: Added a new snippet for Audio Manipulation #171
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.
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
Conversation
Hey, Thank you for your contribution.
After reviewing your PR we've found that all/some snippets you are adding don't match the scope of Quicksnip.
Your snippets acts more like a documentation explaining how to add audio to a page, more than a snippet, providing a function (See guidelines).
I will not close your PR, if you want to fix this snippet so it's more inline with what we expect. If not feel free to close the PR
If no fix is provided within 7-10 days this PR will be closed without further notice
Ah, I see
I'll go ahead and close the PR for this then, since I have another idea for a snippet that actually is just a snippet :)
Description
A new folder under JavaScript was created, audio-manipulation, and a new snippet was created, web-audio-api-starter.md
Type of Change
Checklist
Related Issues
N/A
Closes # N/A
Additional Context
I've always seen Web Audio API in raw JavaScript to be a bit tedious, so I thought to include this and hopefully more snippets in the future.
Screenshots (Optional)
N/A
Click to view screenshots