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

Jump to bottom

feat(Message): add call #10283

Merged
merged 7 commits into from
Jun 8, 2024
Merged

feat(Message): add call #10283

merged 7 commits into from
Jun 8, 2024

Conversation

sdanialraza
Copy link
Contributor

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

@sdanialraza sdanialraza requested a review from a team as a code owner May 18, 2024 14:20
@vercel Vercel
Copy link

vercel bot commented May 18, 2024 •
edited
Loading

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

2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
discord-js ⬜️ Ignored (Inspect) Visit Preview Jun 8, 2024 8:24pm
discord-js-guide ⬜️ Ignored (Inspect) Visit Preview Jun 8, 2024 8:24pm

@vladfrangu vladfrangu added this to the discord.js 14.16 milestone May 19, 2024
@Jiralite Jiralite requested a review from vladfrangu May 20, 2024 17:06
@sdanialraza sdanialraza requested a review from Jiralite May 21, 2024 07:56
@Jiralite Jiralite removed the blocked label Jun 2, 2024
vladfrangu
vladfrangu previously requested changes Jun 2, 2024
View reviewed changes
packages/discord.js/typings/index.d.ts Show resolved Hide resolved
@sdanialraza sdanialraza requested a review from vladfrangu June 2, 2024 21:53
almeidx
almeidx previously requested changes Jun 5, 2024
View reviewed changes
packages/discord.js/src/structures/Message.js Outdated Show resolved Hide resolved
packages/discord.js/src/structures/Message.js Outdated Show resolved Hide resolved
packages/discord.js/src/structures/Message.js Outdated Show resolved Hide resolved
@sdanialraza sdanialraza requested a review from almeidx June 5, 2024 12:24
@WhyAydan WhyAydan mentioned this pull request Sep 28, 2024
Open
@Azzraya Azzraya mentioned this pull request Sep 28, 2024
Open
@ReiseArnor ReiseArnor mentioned this pull request Sep 29, 2024
Open
@whizbangpop whizbangpop mentioned this pull request Oct 3, 2024
Open
@WhyAydan WhyAydan mentioned this pull request Oct 3, 2024
Open
@Azzraya Azzraya mentioned this pull request Oct 3, 2024
Open
@HowlingArcher HowlingArcher mentioned this pull request Oct 4, 2024
Open
@AlexEKinney AlexEKinney mentioned this pull request Oct 4, 2024
Open
@IzzyDotExe IzzyDotExe mentioned this pull request Oct 4, 2024
Merged
@Azzraya Azzraya mentioned this pull request Oct 8, 2024
Open
This was referenced Oct 13, 2024
Open
Open
@jshuntley jshuntley mentioned this pull request Oct 24, 2024
Open
@HowlingArcher HowlingArcher mentioned this pull request Oct 25, 2024
Open
@lkeff lkeff mentioned this pull request Oct 25, 2024
Open
@AlexEKinney AlexEKinney mentioned this pull request Oct 27, 2024
Open
@whizbangpop whizbangpop mentioned this pull request Oct 27, 2024
Open
@WhyAydan WhyAydan mentioned this pull request Oct 28, 2024
Open
@HowlingArcher HowlingArcher mentioned this pull request Nov 1, 2024
Open
@saiteja-madha saiteja-madha mentioned this pull request Nov 2, 2024
Closed
@AlexEKinney AlexEKinney mentioned this pull request Nov 3, 2024
Open
@Adidi1234 Adidi1234 mentioned this pull request Nov 3, 2024
Open
@WhyAydan WhyAydan mentioned this pull request Nov 3, 2024
Open
@Azzraya Azzraya mentioned this pull request Nov 5, 2024
Open
@ClearlyElevated ClearlyElevated mentioned this pull request Nov 6, 2024
Open
@jakejack13 jakejack13 mentioned this pull request Nov 7, 2024
Merged
@AlexEKinney AlexEKinney mentioned this pull request Nov 10, 2024
Open
@WhyAydan WhyAydan mentioned this pull request Nov 10, 2024
Open
@Azzraya Azzraya mentioned this pull request Nov 12, 2024
Open
vladfrangu pushed a commit that referenced this pull request Jan 1, 2025
@sdanialraza @Jiralite @vladfrangu
feat(Message): add call (#10283)
0b0abfa 
* feat(Message): add `call`

* refactor: make `endedAt` a getter

* types: fix `endedAt` return type

* types(Message): add `call` property

* docs: requested changes

---------

Co-authored-by: Jiralite <33201955+Jiralite@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@benjGam benjGam benjGam approved these changes

@almeidx almeidx almeidx approved these changes

@Jiralite Jiralite Jiralite approved these changes

@vladfrangu vladfrangu vladfrangu approved these changes

@iCrawl iCrawl Awaiting requested review from iCrawl iCrawl is a code owner automatically assigned from discordjs/core

@kyranet kyranet Awaiting requested review from kyranet kyranet is a code owner automatically assigned from discordjs/core

@SpaceEEC SpaceEEC Awaiting requested review from SpaceEEC SpaceEEC is a code owner automatically assigned from discordjs/core

Assignees
No one assigned
Labels
api support packages:discord.js semver:minor
Projects
discord.js
Archived in project
Milestone
discord.js 14.16
Development

Successfully merging this pull request may close these issues.
5 participants
@sdanialraza @vladfrangu @Jiralite @almeidx @benjGam