Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: Voice Channel Effect Send #10318

Merged
merged 8 commits into from
Nov 14, 2024
Merged

feat: Voice Channel Effect Send #10318

merged 8 commits into from
Nov 14, 2024

Conversation

Jiralite
Copy link
Member

@Jiralite Jiralite commented Jun 3, 2024

Please describe the changes this PR makes and why it should be merged:

Status and versioning classification:

  • Code changes have been tested against the Discord API, or there are no code changes
  • I know how to update typings and have done so, or typings don't need updating
  • This PR changes the library's interface (methods or parameters added)

Copy link

vercel bot commented Jun 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
discord-js ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 13, 2024 5:02pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
discord-js-guide ⬜️ Ignored (Inspect) Visit Preview Nov 13, 2024 5:02pm

Copy link

codecov bot commented Jul 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 38.16%. Comparing base (e2df0e0) to head (64e7c2f).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #10318   +/-   ##
=======================================
  Coverage   38.16%   38.16%           
=======================================
  Files         239      239           
  Lines       15500    15500           
  Branches     1369     1369           
=======================================
  Hits         5915     5915           
  Misses       9570     9570           
  Partials       15       15           
Flag Coverage Δ
builders 82.43% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Jiralite Jiralite marked this pull request as ready for review October 9, 2024 11:30
@Jiralite Jiralite requested a review from a team as a code owner October 9, 2024 11:30
@Jiralite Jiralite added this to the discord.js 14.17 milestone Oct 9, 2024
Co-authored-by: Danial Raza <danialrazafb@gmail.com>
@Jiralite Jiralite requested a review from sdanialraza October 9, 2024 13:41
@kodiakhq kodiakhq bot merged commit 6775175 into main Nov 14, 2024
25 checks passed
@kodiakhq kodiakhq bot deleted the feat/v-c-s-e branch November 14, 2024 21:00
Jiralite added a commit that referenced this pull request Nov 15, 2024
* feat: Voice Channel Send Effects (#9288)

* feat: add soundboard fields

* chore: address TODO

* docs: volume is a closed interval

* types: use `GatewayVoiceChannelEffectSendDispatchData`

* refactor: prefer getting from cache

* fix: correctly access cache

Co-authored-by: Danial Raza <danialrazafb@gmail.com>

---------

Co-authored-by: Danial Raza <danialrazafb@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants