Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: warn/mute/unmute command #155

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

LightDestory
Copy link
Member

@LightDestory LightDestory commented Dec 21, 2023

Prerequisites

  • I have read and understood the contributing guide.
  • The commit message follows the conventional commits guidelines.
  • Tested the changes locally with a real Telegram bot.
  • Introduced tests for the changes have been added (for bug fixes / features).
  • Docs have been added/updated (for bug fixes / features).
  • I have updated the CHANGELOG.rst file with an overview of the changes made.

Description

This PR is a rework/continuation of the original #142

This PR has been co-authored by @alepiaz

This PR aims to add the following feature:

  • /warn command: You can now warn users and trigger an autoban based on min warn settings. Moreover this feature uses lazy way to detect and delete expired eanrs.
  • mute/unmute commands: You can now mute an user from interactions with the community group.

(If applicable) Issue closed by this PR

As the original PR:

Does this PR introduce a breaking change?

  • Yes
  • No

Python version you are using

Python3.12

Other information

I need testers

@LightDestory
Copy link
Member Author

  1. I don't like the current implementation of some checks, but for now the goal is to ship the feature
  2. I don't know how to fix the fail unit tests, no idea how they are structured

src/spotted/data/user.py Outdated Show resolved Hide resolved
src/spotted/handlers/mute.py Outdated Show resolved Hide resolved
@Helias Helias mentioned this pull request Jan 3, 2024
8 tasks
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.

Comando /warn Implementation of /mute command to mute a user for a defined time
3 participants