Skip to content

Feat/improve sticky#171

Draft
hilmoo wants to merge 6 commits intomainfrom
feat/improve-sticky
Draft

Feat/improve sticky#171
hilmoo wants to merge 6 commits intomainfrom
feat/improve-sticky

Conversation

@hilmoo
Copy link
Member

@hilmoo hilmoo commented Oct 21, 2025

now sticky can not ignore bot

@hilmoo hilmoo requested a review from Copilot October 21, 2025 02:26
@hilmoo hilmoo requested a review from Iqrar99 as a code owner October 21, 2025 02:26
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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_bot boolean column to the sticky message database schema
  • Updated sticky message handling logic to skip reposting when bot messages are received and ignore_bot is enabled
  • Added ignore_bot parameter 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.

@hilmoo
Copy link
Member Author

hilmoo commented Oct 21, 2025

pyright error #170

@hilmoo hilmoo requested a review from Copilot October 21, 2025 02:37
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@hilmoo hilmoo requested a review from Copilot October 21, 2025 02:40
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@hilmoo hilmoo marked this pull request as draft October 21, 2025 03:44
@Iqrar99
Copy link
Member

Iqrar99 commented Oct 21, 2025

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 AllowedMentions class whenever sending a message and set it all to False.

@hilmoo
Copy link
Member Author

hilmoo commented Oct 21, 2025

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 AllowedMentions class whenever sending a message and set it all to False.

yeah i forgot that case

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants