Skip to content
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

types: fix missing types for mentionable options #8443

Merged
merged 3 commits into from Aug 10, 2022
Merged

types: fix missing types for mentionable options #8443

merged 3 commits into from Aug 10, 2022

Conversation

janparisek
Copy link
Contributor

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

  • Add ApplicationCommandRoleOptionData interface
  • Add ApplicationCommandRoleOption interface
  • Add ApplicationCommandUserOptionData interface
  • Add ApplicationCommandUserOption interface
  • Add ApplicationCommandMentionableOptionData interface
  • Add ApplicationCommandMentionableOption interface
  • Update ApplicationCommandSubCommandData.options union
  • Update ApplicationCommandOptionData union
  • Update ApplicationCommandOption union

Fixes #8440

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)

- Add `ApplicationCommandRoleOptionData` interface
- Add `ApplicationCommandRoleOption` interface
- Add `ApplicationCommandUserOptionData` interface
- Add `ApplicationCommandUserOption` interface
- Add `ApplicationCommandMentionableOptionData` interface
- Add `ApplicationCommandMentionableOption` interface
- Update `ApplicationCommandSubCommandData.options` union
- Update `ApplicationCommandOptionData` union
- Update `ApplicationCommandOption` union
@vercel
Copy link

vercel bot commented Aug 5, 2022

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

Name Status Preview Updated
discord-js ✅ Ready (Inspect) Visit Preview Aug 10, 2022 at 4:28PM (UTC)

@Jiralite Jiralite added this to the discord.js v14.2 milestone Aug 5, 2022
@vercel vercel bot temporarily deployed to Preview August 5, 2022 16:56 Inactive
kyranet
kyranet previously requested changes Aug 7, 2022
packages/discord.js/typings/index.d.ts Outdated Show resolved Hide resolved
Co-authored-by: A. Román <kyradiscord@gmail.com>
@vercel vercel bot temporarily deployed to Preview August 9, 2022 22:32 Inactive
@iCrawl iCrawl requested a review from kyranet August 10, 2022 08:11
@vercel vercel bot temporarily deployed to Preview August 10, 2022 16:28 Inactive
@codecov
Copy link

codecov bot commented Aug 10, 2022

Codecov Report

Merging #8443 (f3fcb9d) into main (38275fc) will increase coverage by 5.52%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #8443      +/-   ##
==========================================
+ Coverage   86.41%   91.93%   +5.52%     
==========================================
  Files          87       75      -12     
  Lines        8286     6856    -1430     
  Branches     1104     1018      -86     
==========================================
- Hits         7160     6303     -857     
+ Misses       1083      511     -572     
+ Partials       43       42       -1     
Flag Coverage Δ
builders 100.00% <ø> (ø)
collection 100.00% <ø> (ø)
proxy 73.28% <ø> (ø)
rest 92.53% <ø> (ø)
utilities 100.00% <ø> (ø)
voice 64.08% <ø> (ø)
ws ?

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

Impacted Files Coverage Δ
packages/ws/src/utils/constants.ts
packages/ws/src/ws/WebSocketShard.ts
packages/ws/src/ws/WebSocketManager.ts
packages/ws/src/utils/utils.ts
.../src/strategies/sharding/SimpleShardingStrategy.ts
...trategies/context/SimpleContextFetchingStrategy.ts
...src/strategies/context/IContextFetchingStrategy.ts
...trategies/context/WorkerContextFetchingStrategy.ts
...es/ws/src/strategies/sharding/IShardingStrategy.ts
packages/ws/src/utils/IdentifyThrottler.ts
... and 2 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@iCrawl iCrawl merged commit 452e94f into discordjs:main Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Missing types for application command mentionable options
5 participants