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
feat(discord-utilities): interaction option resolver #515
feat(discord-utilities): interaction option resolver #515
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of things and as said above this needs some proper testing. I also want to go through the code and do some syntax changing to more match the sapphire style (I.e. we don't really use key in object
generally)
packages/discord-utilities/src/lib/CommandInteractionOptionResolver.ts
Outdated
Show resolved
Hide resolved
packages/discord-utilities/src/lib/CommandInteractionOptionResolver.ts
Outdated
Show resolved
Hide resolved
Would you like me to write unit tests? |
packages/discord-utilities/src/lib/CommandInteractionOptionResolver.ts
Outdated
Show resolved
Hide resolved
packages/discord-utilities/src/lib/CommandInteractionOptionResolver.ts
Outdated
Show resolved
Hide resolved
ae5dfdb
to
9fb8dd7
Compare
Also rebased |
9fb8dd7
to
d5a932d
Compare
* feat(discord-utilities): command interaction option resolver * fix(CommandInteractionOptionResolver): proper constructor type * chore: consistent spacing * fix(CommandInteractionOptionResolver): actually make required arg optional * chore: add tests * chore: hey, look over there! permalink: http://whatthecommit.com/87cf7faa6a18dc6d7232dea310109684 * chore: rename Co-authored-by: Jeroen Claassens <support@favware.tech>
No description provided.