Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds functionality to allow sticky messages to optionally ignore messages sent by bots. A new ignore_bot boolean field is added to the sticky message system with a default value of TRUE, meaning bot messages will be ignored by default.
Key Changes
- Added
ignore_botboolean column to the sticky message database schema - Updated sticky message handling logic to skip reposting when bot messages are received and
ignore_botis enabled - Added
ignore_botparameter to add and edit sticky message commands
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| bot/data/db.sql | Adds ignore_bot column to scheduled_message table with default TRUE |
| bot/cogs/sticky.py | Implements ignore_bot logic in message handler and updates CRUD operations to support the new field |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
pyright error #170 |
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
I've been considering adding a feature that prevents the bot from pinging users or roles when mentioning them. This way, we can safely tag roles or users without triggering a ping. To achieve this, we should use the |
yeah i forgot that case |
now sticky can not ignore bot