-
Notifications
You must be signed in to change notification settings - Fork 3
Feature Improvements #2
Comments
Could you elaborate some of these points a bit more? I'll go over some of these myself with the immediate remarks I have, but please expand on all of them.
This is a tricky one to implement, The way I see it there's two routes to implement this:
None of those are particularly appealing to me. Perhaps you could clarify the need of this feature, and see if we can't find an alternative approach.
This has been asked a few times already in Kord itself. I'm of the opinion myself that defaults can be perfectly simulated by users via extension functions. Users can implement their own, for example,
A good feature to request IMO. Something I personally wanted to add to Kord in some way or form myself. Regarding the optional escaping. I had though this was mostly solved with the |
Main use for this which I've found quite helpful in other bots is for correcting small errors whilst processing commands.
Yep that makes a lot of sense. I'll remoe that from the list.
Yep, only Discord's subset of markdown should be needed. Also with the escaping I think that'd be a good idea to include, but as false by default in the methods. Main reason is that, for example, certain names with underscores don't italicise a join message for example. Up to you of course. |
I recently stumbled across this and needed this for some personal projects, I was working on this since yesterday. Almost done, but currently the caching for data between invocation of edited command and last ran command (as an example to track the message we've sent on the first/prior run) is not properly handled due to Type inference failed by lambda return type. So, I'd PR for this feature as soon as this lib synchronize with Kord 0.7.x and the mentioned issue is fixed. |
Suggestions
The text was updated successfully, but these errors were encountered: