alexandercolen / annie-may-discord-bot Goto Github PK

Quick check for their Discord ID from the database.

Include how to test it yourself before launching, how to add a new module/command and how to request it to be put live.

The string.IsNullOrEmpty() method exists, so replace all of those with that.

Display their information as found on Anilist, and add their most famous animation works.

Use reactions to vote for the options.

For example:

public class MongoDBUtility
    {
        public IUser FindDiscordUserInChannel(IChannel channel, ulong userId)
        {
            return channel.GetUserAsync(userId).Result;
        }
    }

Create image of the mascot holding up a blank sign and allow the user to type something and that to then be placed on top of the sign and returned.

Try to take as many different alphabets into account, not just latin alphabet.

And prompt them to start the setup process

New category

MongoDB is not ideal for expanding, especially the caching of data, and is causing blockage for other issues. ( #3 )

Migrate the data to PostgreSQL and include DB creation scripts within the code base.

Perhaps couple this with the move to a Docker container.

Add format below this line for GraphQL fetch to fix it.

Add a check for MediaType.Movie and remove the 'per episode' text.

Go through all the code in the project and make sure that all the variables are named in the same way.

Main culprit is the Models.DiscordUser class.

Also add some documentation guidelines to the wiki on how to adhere to these standards.

Once a single user has registered, it means that UUID 000000000000000000000000 is taken, and therefore when the next person tries it creates a duplicate ID error.

Clear and concise one-liners as summary description and map out all the parameters and return values.

Might want to get rid of:
[Summary("This is a simple method summary.")]
and replace with:

/// <summary>
/// This is a more descriptive method summary.
/// </summary>
/// <param name="aParameter">I am the method parameter.</param>
/// <returns>I am the return value.</returns>

Make sure that every method is at least covered by a summary, ideally add comments within code with reasons for a loop, if, and return statement.

Cut description before adding it here:

Take the first 2045 characters and add ... to indicate that there was more.

**Make sure to check if the part that was cut was not a spoiler, and if it is make sure to cut 5 chars and add ...|| instead.

Should default to random media from Anilist.

Multiple possible overrides:

• Specify anime or manga. Will become its own issue eventually™.
• Random entry from someone's list. Will become its own issue eventually™.
• Random entry from the user's Planned list.

Should be similar to Character & Studio, so reuse as many elements as you can from that.

Probably best to refactor some code for less duplication as well?

Minimum functionality:

• Add database support for MAL
• Merge user scores of both MAL & Anilist when fetching media data
• Have users add MAL on server setup
  • Make sure to also ask for MAL or anilist preference

Change status from a list of predetermined statuses every X seconds.

Make sure to keep the ~help before the custom status!

Randomly roll a die or flip a coin, allow user overrides to roll between certain numbers.

Potentially roll multiple times?

Most likely caused by a staff alias that contains a grave accent ( ` ).

Test staff that breaks it: Kanye West

Turns out the implementation for #35 wasn't working.

Also allow just using their discord ID, by checking for certain length of the int parameter.

It takes seconds as of now with only 10 entries because of all the loops required.

Store them in the database.

Solve it using the Levenshtein utility just like was done for anime searching.

Steps to resolve this issue:

1. Fetch a list of characters based on the search criteria.
2. Run characters with all their aliases through the Levenshtein algorithm.
3. Return the best response to the user.

Currently being solved using Levenshtein algorithm, but the result is not as desired.
Sometimes the searched query is not even close to the actual entry that is known to exist that we want to find.

Caused by the looking up of users' lists and trying to parse their media status.
Error converting value {null} to type 'AnnieMayDiscordBot.Enums.Anilist.MediaStatus'. Path 'data.MediaListCollection.lists[0].entries[42].media.status', line 1, position 12942.

Fixing #41 by refactoring the way it is handled may fix this issue.

There needs to be 2 newlines before and after regex.

Needs to be fixed for these:

Display their information as found on Anilist.

Following error gets thrown:
Invalid context type. Expected SocketCommandContext, got CustomCommandContext.

The following should at least be in there:

• Purpose
• Written by/in
• GitHub link

Handle it like Season is handled, by changing 0 to ?.

Example of problem:

Will make it more clear how to add more commands for other users and give more ways to add fluff to the bot, such as rewriting the :3 thing.

Calculate affinity between two users upon request and across every registered user in the server.

Can base the calculation on how it was done for this Discord bot:
https://github.com/jerome-ceccato/andre/blob/1ac2500605c18e9da6cc044793af157d8d40fc73/commands/mal.py#L598-L654

Pretty much identical to #26, so a similar fix should be implemented.

Test staff that breaks it: Kanye West

Add some fancy/proper markdown formatting to make it look acceptable.

It links towards an ID of 0, meaning that the idMal attribute is not being fetched from the GraphQL API.

character.name.native can be empty, so has to be checked before this line.

Possible that the bot is getting overloaded because of requests, in which case a timeout should be added between commands.

Changing this to int? should resolve the issue.

Only show distribution of their scores. Can make use of the ScoreDistribution class (Annie/Models/Anilist/ScoreDistribution.cs)

Allow custom overloading using >, <, = and != including combinations of these.

Currently a hard-coded list. Yuck.