search

underctrl-io / commandkit

Only focus on what matters - Let CommandKit handle your commands and events in your Discord.js projects!
https://commandkit.js.org
MIT License
86 stars 11 forks source link

Command builders from CommandKit-Helper #18

Closed FearlessNite345 closed 11 months ago

FearlessNite345 commented 11 months ago

So in this PR i have added the command builders to the base command kit which can be found inside my current npm package commandkit-helper i have also included the documentation changes to support for the builders and have run my local testing to make sure this all works on my end and have found no errors upon doing so

netlify[bot] commented 11 months ago

Deploy Preview for commandkit ready!

Name Link
Latest commit 6e1324e9f3dedba2760a228ad1eaf92e0f105523
Latest deploy log https://app.netlify.com/sites/commandkit/deploys/65257d06953b5c00082ae8b7
Deploy Preview https://deploy-preview-18--commandkit.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

NotM1Dev commented 11 months ago

@notunderctrl You planning to add builders? I don't think they're any better than how CommandKit is currently. The most builders can be used for is probably slash command or context menu command builders. I've realized these can be really helpful for beginners lol

notunderctrl commented 11 months ago

Hi @FearlessNite345

First of all thank you so much for contributing to CommandKit. Builders is definitely something I want to introduce to CommandKit. However, the codebase is currently not in the state I'd like it to be. There's huge refactoring that I'd like to do before introducing builders.

With that being said, your contribution is not going to waste because I'll be using this as a base to introduce builders once the refactoring is complete.

Thanks once again.