Update the help setup command.

cogs/commands/help.py

@@ -1,29 +1,21 @@
-"""
+"""Commands for helping a user."""
-Commands for helping a user.
-"""
 
 import nextcord
 from nextcord.ext import commands
 
 
 class HelpCog(commands.Cog, name="Help commands."):
-    """
+    """Class holds commands for getting help."""
-    Class holds commands for getting help.
-    """
 
     def __init__(self) -> None:
-        """
+        """Default constructor for HelpCog class. Does nothing."""
-        Default constructor for HelpCog class. Does nothing.
-        """
 
     @nextcord.slash_command()
     async def help(self, interaction: nextcord.Interaction):
-        """
+        """Parent slash command for help. Never called."""
-        Parent slash command for help. Never called.
-        """
 
     @help.subcommand(description="Describes how to get started with Agon.")
-    async def setup(self, interaction: nextcord.Interaction):
+    async def setup(self, interaction: nextcord.Interaction) -> None:
         """
         Sends a message describing how to get started with Agon.
 
@@ -31,15 +23,13 @@ class HelpCog(commands.Cog, name="Help commands."):
             interaction (nextcord.Interaction): Interaction that invoked the command.
         """
         # I hate this but don't want indents in the string.
-        # pylint: disable=line-too-long
-        help_str: str = """Welcome to Agon! Three simple steps are needed to get started.
-1.) Set a channel to recieve submission verifications with "/set verify_channel."
-This channel is used to allow moderators to check if a submission is real/valid.
 
-2.) Set a channel for submissions to be posted in with "/set submission_channel."
+        help_str: str = "Welcome to Agon! Three simple steps are needed to get started."
-This channel will be where all user submission messages are sent.
+        help_str += '1.) Set a channel to receive submission verifications with "/set verify_channel."\n'
-
+        help_str += "This channel is used to allow moderators to check if a submission is real/valid.\n\n"
-3.) Set up your highscore tables! Use "/create group" to first create a group message in the channel you would like and then use "/create category" to create a category in that group message!"""
+        help_str += '2.) Set up your highscore tables! Use "/create group" to first create a group message in the'
-        # pylint: enable=line-too-long
+        help_str += 'channel you would like and then use "/create category" to create a category in that group message!'
+        help_str += "\n\n"
+        help_str += '3.) Users can then submit scores with the "/submit" command.'
 
         await interaction.response.send_message(content=help_str)

cogs/commands/set.py

@@ -103,7 +103,9 @@ class SetCog(commands.Cog, name="Set command."):
                 f"Category pad length set to {length} for {group.GroupName}", ephemeral=True
             )
 
-    @set.subcommand(description="Registers the channel for submission messages.")
+    @set.subcommand(
+        description="Registers the channel for submission messages. Currently this information is deprecated."
+    )
     async def submission_channel(
         self,
         interaction: nextcord.Interaction,

cogs/submission/verification_view.py

@@ -1,6 +1,10 @@
 """View for verification buttons."""
 
 import nextcord
+from sqlalchemy import select
+from sqlalchemy.orm import Session
+
+from database_setup.sql_data_classes import Submission, engine
 
 # pylint: disable=import-error
 from utilities.utils import submit_score
@@ -20,27 +24,24 @@ class VerificationView(nextcord.ui.View):
     @nextcord.ui.button(label="Approve", style=nextcord.ButtonStyle.green, custom_id="approve-button-submission")
     async def approve_button(self, button: nextcord.ui.Button, interaction: nextcord.Interaction) -> None:  # noqa: ARG002
         """Button for approving score submission."""
-        for embed in interaction.message.embeds:
+        with Session(engine) as session:
-            content = embed.to_dict()
+            stmt = (
+                select(Submission)
+                .where(Submission.GuildId == interaction.guild_id)
+                .where(Submission.MessageId == interaction.message.id)
+            )
 
-            for field in content["fields"]:
+            submission = session.scalar(stmt)
-                if field["name"] == "GroupId":
-                    group_id = int(field["value"])
-                elif field["name"] == "CategoryId":
-                    category_id = int(field["value"])
-                elif field["name"] == "Score":
-                    score = field["value"]
-                elif field["name"] == "User Name":
-                    user = field["value"]
 
-        # TODO: rework to get the information from database instead
+            if submission is None:
+                await interaction.response.send_message("Could not find submission data in database.", ephemeral=True)
+                await interaction.message.delete()
+                return
 
-        try:
+            await submit_score(
-            await submit_score(interaction, group_id, category_id, user, score)
+                interaction, submission.GroupId, submission.CategoryId, submission.UserName, submission.Score
+            )
             content = interaction.message.content
             content += f"\nApproved by <@{interaction.user.id}>."
             # remove the bot information embed. it should always be the first one.
-            await interaction.message.edit(content=content, view=None, embed=interaction.message.embeds[0])
+            await interaction.message.edit(content=content, view=None)
-        except UnboundLocalError:
-            await interaction.send("Instruct user to submit with all fields.")
-            await interaction.message.edit(view=None)

highscores_database.sql

@@ -4,7 +4,7 @@ use highscores;
 CREATE TABLE Guilds(
    GuildId bigint unsigned not null unique,
    GuildName varchar(95) not null,
-   SubmissionChannel bigint unsigned,
+   SubmissionChannel bigint unsigned, -- deprecated, is not used anymore.
    VerificationChannel bigint unsigned,
    Active boolean DEFAULT true,
    PRIMARY KEY (GuildId)