Comments (4)
kangalio
commented on August 17, 2024
2
from poise.
kangalio
commented on August 17, 2024
True, that should be changed. I'm open for PRs I don't have much time myself
from poise.
kangalio
commented on August 17, 2024
What do you think of the println!() calls in the default on_error function? This function is intended as a starting building block for new users, hence I decided to use println because that's easiest and most hassle-free since it doesn't require a logging library set up. What do you think?
from poise.
Erik1000
commented on August 17, 2024
I think it should use the logging/tracing library as well. Setting up a logger/tracer is a one liner and serenity already depends and uses the tracing crate. Maybe it would be a good idea to have an example logging/tracing setup in the docs for new users but other then that I think it’s fine.
from poise.
Related Issues (20)
- Add support for integer and double option choices. HOT 8
- Add serenity's `CommandId` to slash commands created with poise. To reference it in the help menu HOT 1
- Add fallback message handler HOT 3
- Add an optional way to prevent docstring consumption HOT 1
- Are all examples meant to be listed? HOT 2
- Atomics - Should this be fetch_add? HOT 2
- the trait bound `poise::serenity_prelude::Context: AsRef<poise::serenity_prelude::Context>` is not satisfied HOT 9
- Handle events and still using the poise context HOT 1
- Events not firing on latest `serenity-next` HOT 1
- single-command help shows `<prefix>` for bot with no prefix
- Cannot use any parameters in context menu function HOT 3
- PopArgumentAsync isn't real HOT 1
- ArgumentParse errors from poise::command are not sanitized HOT 1
- Allow to provide a set of Team Roles, to be added to the owner array
- Rename modal title when executing from a struct that derives poise::Modal? HOT 1
- Adding attachments to initial interaction response HOT 1
- Cannot use *Id arguments like the full counterparts in prefix commands
- `Context::reply` uses old builder syntax in documentation
- CreateReply on `next` should use `impl Into<Cow<'a, str>>` instead of `impl Into<String>`
- Allow for using macros as values in `poise::command` attribute
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from poise.