Skip to content

Conversation

@petabyte-imo
Copy link
Contributor

@petabyte-imo petabyte-imo commented Sep 4, 2024

Description

This adds a temp ban command and some stuff for the db that helps do that, made for #426

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Other: (write here)

Guidelines

  • My code follows the style guidelines of this project (formatted with Ruff)

  • I have performed a self-review of my own code

  • I have commented my code, particularly in hard-to-understand areas

  • I have made corresponding changes to the documentation if needed

  • My changes generate no new warnings

  • I have tested this change

  • Any dependent changes have been merged and published in downstream modules

  • I have followed all of these guidelines.

How Has This Been Tested? (if applicable)

It has been tested in the ATL dev server

Screenshots (if applicable)

Please add screenshots to help explain your changes.

Additional Information

DONT MERGE IT

Summary by Sourcery

Add a temporary ban command and an automated unban task to manage temporary bans in the system. Enhance the database controller to support retrieving all guilds and managing temporary ban states.

New Features:

  • Introduce a temporary ban command that allows moderators to ban a member for a specified duration with options for reason, purge days, and silent execution.
  • Implement a scheduled task to automatically unban users whose temporary bans have expired.

Enhancements:

  • Add a method to retrieve all guilds from the database, enhancing the guild management capabilities.

@ghost ghost marked this pull request as draft September 5, 2024 10:31
@ghost ghost changed the title Temp Ban command [DONT MERGE] Temp Ban command & Unban Task Sep 5, 2024
@ghost ghost marked this pull request as ready for review September 5, 2024 13:52
@sourcery-ai

This comment was marked as duplicate.

sourcery-ai[bot]

This comment was marked as resolved.

@ghost ghost removed the status: needs testing label Sep 5, 2024
@ghost
Copy link

ghost commented Sep 5, 2024

LGTM - Ready for merge

Copy link
Member

@electron271 electron271 left a comment

Choose a reason for hiding this comment

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

as per the review by @wlinator, please in the future use the github review system

@electron271 electron271 merged commit 693d371 into allthingslinux:main Sep 6, 2024
@sentry
Copy link

sentry bot commented Sep 6, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ DataError: The column Case.case_tempban_expired does not exist in the current database. tux.database.controllers.case in get_expired_te... View Issue
  • ‼️ TableNotFoundError: The table public.Case does not exist in the current database. tux.database.controllers.case in get_expired_te... View Issue

Did you find this useful? React with a 👍 or 👎

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants