[FEATURE] Forum Showcase (#264)

* add: forum showcase table

* add: forum showcase db

* add: forum showcase cog

* chore: format

* add: relativedelta module

* chore: rename migration file

* add: forum showcase view

* add: forum showcase cog

* add: forum showcase data

* fix: stop is not async

* fix: migrate_db

* chore: remove view

* feat: add showcase snd scheduling

* add: autocomplete
fix: schedule

* fix: schedule

* add: id field

* fix: add and delete function

* feat: add and delete command

* remove: status field

* add: toggle command

* fix: refreshloopinterval

* add: list command

* add: config command

* Remove forum showcase view

* Update config and add command

* fix: config command

* Refactor scheduling

* add: log if showcase is disabled

* Disable showcase by default

* change config name

* Revert old migration file names

* Start task forum showcase task if not running

* Exclude archived threads in showcase
Fix schedule timezone

* fix: showcase not running

* use utc timezone

* Convert UTC+8 to UTC+0

* remove unecessary timezone.utc in datetime.now function

* Fix random selection of threars

* fix: showcase not running

* Catch negative second

* Set check to zero

* Set interval default to weekly

* Add weekday field

* Fix migrate error

* add: weekday field

* Add weekday select

* add: scheduler

* add: weekday field

* add: forum showcase views

* add: logger in forum showcase db

* add: weekday configuration for forum showcase cog

* fix: error running poetry, unittest version not found

* add: typehint for migration

* refactor: currency converter intialization of symbols

* fix: schedule

* chore: remove asyncio

* fix: selected configuration not syncing

* add: log when schedule is changed

* chore: remove log

* fix: format date

* fix: weekly schedule

* fix: parsing time

* fix: new schedule not updating

* chore: separate next_run calculation

* fix: set scheduled time to bot state

* chore: calculate next run directly

* chore: set the state inside the select function

* add: interval selection

* fix: interval selection

* fix: timezone issue

Author: mystique09

migrations/16_string_matching_improvement.sql

@@ -1,2 +1,2 @@
 alter table pph_auto_responses
-add matching_type varchar(255) default 'lenient' not null;
+add IF NOT EXISTS matching_type varchar(255) default 'lenient' not null;

migrations/19_thread_showcase.sql

@@ -8,7 +8,7 @@ CREATE TABLE IF NOT EXISTS pph_forum_showcase (
     id SERIAL PRIMARY KEY,
     target_channel BIGINT NOT NULL,
     schedule TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP,
-    interval forum_showcase_interval NOT NULL DEFAULT 'daily',
+    interval forum_showcase_interval NOT NULL DEFAULT 'weekly',
     created_at TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP,
     updated_at TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP
 );

migrations/20_add_weekday_field.sql

@@ -0,0 +1,6 @@
+--
+--depends: 19_thread_showcase
+
+CREATE TYPE forum_showcase_weekday AS ENUM ('Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday', 'Saturday', 'Sunday');
+
+ALTER TABLE IF EXISTS pph_forum_showcase ADD COLUMN weekday forum_showcase_weekday NOT NULL DEFAULT 'Monday';

poetry.lock

@@ -5,10 +5,10 @@ description = ""
 authors = ["Jedddy <jedbalita25@gmail.com>"]
 readme = "README.md"
 packages = [
-    {include = "src"},
-    {include = "bot", from = "src"},
-    {include = "cogs", from = "src"},
-    {include = "*", from = "src/bot"}
+    { include = "src" },
+    { include = "bot", from = "src" },
+    { include = "cogs", from = "src" },
+    { include = "*", from = "src/bot" },
 ]
 
 [tool.poetry.dependencies]

pyproject.toml

@@ -1,5 +1,7 @@
 import os
 
+from yoyo.backends.base import DatabaseBackend
+
 from src.utils.logging.logger import BotLogger
 from src.bot.config import Database, Config, get_config
 from src.utils.logging.discord_handler import DiscordHandler
@@ -23,7 +25,13 @@ class ProgPhil(Bot):
     bot_logger: BotLogger
     pool: Pool
 
-    def __init__(self, pool: Pool, cfg: Config, bot_logger: BotLogger, **kwargs):
+    def __init__(
+        self,
+        pool: Pool,
+        cfg: Config,
+        bot_logger: BotLogger,
+        **kwargs,
+    ):
         bot_cfg = cfg.bot
         super().__init__(
             **kwargs,
@@ -45,11 +53,11 @@ async def on_ready(self) -> None:
         logger_config = self.config.logger
         log_channel = self.get_channel(logger_config.log_channel)
 
-        discord_handler = DiscordHandler(log_channel)
+        discord_handler = DiscordHandler(log_channel)  # type: ignore
         bot_logger.add_handler(discord_handler)
 
         logger = bot_logger.get_logger()
-        logger.info(f"{self.user.display_name} running.")
+        logger.info(f"{self.user.display_name} running.")  # type: ignore
 
         self.logger = logger
 
@@ -109,13 +117,12 @@ def migrate_db(db: Database, logger: Logger) -> None:
     logger.info(f"Starting database migration with URL: {url}")
 
     try:
-        backend = get_backend(url)
+        backend: DatabaseBackend = get_backend(url)
         migrations = read_migrations("./migrations/")
+        to_apply = backend.to_apply(migrations)
 
-        with backend.lock():
-            to_apply = backend.to_apply(migrations)
-            logger.info(f"Found {len(to_apply)} migrations to apply")
-            backend.apply_migrations(to_apply)
+        logger.info(f"Found {len(to_apply)} migrations to apply")
+        backend.apply_migrations(to_apply)
         logger.info("Migration completed successfully")
     except Exception as e:
         logger.error(f"Error during migration: {str(e)}")
@@ -139,7 +146,7 @@ async def main():
 
     migrate_db(db_config, logger.get_logger())
 
-    bot = ProgPhil(pool, config, logger)
+    bot = ProgPhil(pool, config, logger)  # type: ignore
     await bot.launch()