flybywiresim / discord-bot-utils Goto Github PK

Feature type

Utility or complex functionality

Description

A voting system. Features to be discussed.

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

benw8484

Bug type

Utility or complex functionality issue

Describe the bug

Double plus needs fixing. A check needs to be added to strip an extra plus. The command defaults to + with the offset, and it's adding a second if the user specifies + when setting an offset

Expected behavior

There should only be one + shown regardless of the user inputting 1 or +1 (or any number) in the offset

Steps to reproduce

Run the zulu command with an offset of +1 (or any number)

References (optional)

Additional info (optional)

No response

Discord Username (optional)

benw8484

Feature type

Utility or complex functionality

Description

Similar to the PR command, but for github issues instead

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

straks

Bug type

Utility or complex functionality issue

Describe the bug

When a message is longer than 1024 characters and is edited or deleted, the bot will crash due to the 1024 character limitation per field.

Expected behavior

It should be changed to limit the fields of old message, edited message and deleted message to 1000 characters and then link the message instead.

For deleted messages, maybe another embed could be sent to show the whole message.

Steps to reproduce

Edit or delete a message longer than 1024 characters

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

benw8484

Bug type

Utility or complex functionality issue

Describe the bug

Just need to change the logging in the scamLogs handler to make it clearer that it is from scam logs

Logger.info('Starting Infraction process');

            const newInfraction = {
                infractionType: 'ScamLog',
                moderatorID: msg.client.user.id,
                reason: `Message content: ${msg.content.toString()}`,
                date: new Date(),
                infractionID: new mongoose.Types.ObjectId(),
            };

            let userData = await Infraction.findOne({ UserID: msg.author.id });

            log(userData);

For example, just prefix with scamLogs

(Other commands may benefit from this, I just saw this when flicking through logs)

Expected behavior

Prefix the logs

Steps to reproduce

Run the command and check logs

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

benw8484

Bug type

Wrong data or information

Describe the bug

Content needs to be changed to reflect the correct use of the channel.

Expected behavior

Content should read:

We've opened our #flight-school channel for any questions you have pertaining to the operation of the A32NX.

Steps to reproduce

Run the /welcome command

References (optional)

Additional info (optional)

No response

Discord Username (optional)

benw8484

Bug type

Wrong data or information

Describe the bug

When a long message of over 1024 characters contains @everyone and gets caught by scam logs, it will restart the bot because of an exception with core content:

embeds[0].fields[2].value[BASE_TYPE_MAX_LENGTH]: Must be 1024 or fewer in length.

More details:
https://logs.betterstack.com/team/205992/tail?q=&s=662625&a=1707528913978168.126000002

Expected behavior

Bot does not restart on large messages being captured as part of scam logging

Steps to reproduce

1. Run the bot
2. Post a message over 1024 characters long, with @everyone in it.

References (optional)

Similar issue as #19

Additional info (optional)

No response

Discord Username (optional)

straks

Feature type

Moderation feature

Description

A utility available to moderators which lists all members of a role.

Consider that a role can have 100s of members, so pagination will be a must

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

straks

Feature type

Utility or complex functionality

Description

The command table generation could be separated into a function in lib.

This function can then be called in the /commandTable command as well as a scheduler job. This will make sure that the command table is up to date. Only needs to be ran once a day.

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

benw8484