If the queue channel, log channel or top_ten channels are deleted, they should be removed from the database. As this grows, we are very likely to have an excess amount of dead channels

Valorant

Once a game is found, the lobby queue should select a map from below at random to be played. This can be another embed with the map image and "Map to be played" description. This will be optional.

Valorant Maps:

Haven - https://media.valorant-api.com/maps/2bee0dc9-4ffe-519b-1cbd-7fbe763a6047/splash.png
Split - https://media.valorant-api.com/maps/d960549e-485c-e861-8d71-aa9d1aed12a2/splash.png
Ascent - https://media.valorant-api.com/maps/7eaecc1b-4337-bbf6-6ab9-04b8f06b3319/splash.png
Icebox - https://media.valorant-api.com/maps/e2ad5c54-4114-a870-9641-8ea21279579a/splash.png
Fracture - https://media.valorant-api.com/maps/b529448b-4d60-346e-e89e-00a4c527a405/splash.png
Pearl - https://media.valorant-api.com/maps/fd267378-4d1d-484f-ff52-77821ed10dc2/splash.png
Lotus - https://media.valorant-api.com/maps/2fe4ed3a-450a-948b-6d6b-e89a78e680a9/splash.png
Bind - https://media.valorant-api.com/maps/2c9d57ec-4431-9c5e-2939-8f9ef6dd5cba/splash.png
Breeze - https://media.valorant-api.com/maps/2fb9a4fd-47b8-4e7d-a969-74b4046ebd53/splash.png

We will then send a map ban link like op.gg. Please select Bo1 (1Star on the website)
https://www.mapban.gg/en/ban/valorant/competitive

With this teams can ban maps and select which sides they want to start on, Attacking or Defending.

Overwatch

Overwatch Game modes + Maps:
Once lobby has started, select ONE game mode at random then ONE random map within the game mode.

Game modes + Map combinations

https://overfast-api.tekrop.fr/maps
Control:

• Busan - https://overfast-api.tekrop.fr/static/maps/busan.jpg
• Ilios - https://overfast-api.tekrop.fr/static/maps/ilios.jpg
• Lijiang Tower - https://overfast-api.tekrop.fr/static/maps/lijiang.jpg
• Nepal - https://overfast-api.tekrop.fr/static/maps/nepal.jpg
• Oasis - https://overfast-api.tekrop.fr/static/maps/oasis.jpg

Escort:

• Dorado - https://overfast-api.tekrop.fr/static/maps/dorado.jpg
• Junkertown - https://overfast-api.tekrop.fr/static/maps/junkertown.jpg
• Circuit Royal - https://overfast-api.tekrop.fr/static/maps/circuit_royal.jpg
• Rialto - https://overfast-api.tekrop.fr/static/maps/rialto.jpg
• Route 66 - https://overfast-api.tekrop.fr/static/maps/route_66.jpg
• Shambali Monastery (new) - https://overfast-api.tekrop.fr/static/maps/shambali.jpg
• Watchpoint Gibratar https://overfast-api.tekrop.fr/static/maps/gibraltar.jpg

Hybrid:

• Blizzard World - https://overfast-api.tekrop.fr/static/maps/blizzard_world.jpg
• Eichenwalde - https://overfast-api.tekrop.fr/static/maps/eichenwalde.jpg
• King’s Row - https://overfast-api.tekrop.fr/static/maps/kings_row.jpg
• Midtown - https://overfast-api.tekrop.fr/static/maps/midtown.jpg
• Paraíso - https://overfast-api.tekrop.fr/static/maps/paraiso.jpg
• Numbani - https://overfast-api.tekrop.fr/static/maps/numbani.jpg
• Hollywood - https://overfast-api.tekrop.fr/static/maps/hollywood.jpg

Push:

• Colosseo - https://overfast-api.tekrop.fr/static/maps/colosseo.jpg
• New Queen Street - https://overfast-api.tekrop.fr/static/maps/new_queen_street.jpg
• Esperança - https://overfast-api.tekrop.fr/static/maps/esperanca.jpg

Let's select game modes and maps at random at first. We can refine this to add map voting or similar later on.

Should be released with - #15

If the queue is in a ready check state and someone is not available, the /admin reset [member] will not work. It will remove them from the queue yes but the queue will stay in a ready check.

Even worse, if they come back after the /admin reset [member] has been run, and they click ready, the game will start but they won't be in the lobby.

Solution:
Given a user has not readied up
When the /admin reset [member] command has been run
Then they are properly removed from the queue
And the queue returns to a queue state

Temp workaround:
Wait for the timer to run out, or start an entirely new queue without them

1. Someone runs /start
2. A game is found and begun
3. Bot automatically sends another queue by itself.

So save time, lets leave out the top.gg vote for now.

This will keep a steady flow of queues

I want people to have more flexibility with the bot; many features can/should be toggled on and off so all servers can fine-tune the bot to their needs. For example, the MVP vote - can easily be an on/off feature! Also, the admin commands are all for Administrators only. We can be flexible here and move it down to the "manage server" role and then specific commands only Admins can run. Or even more flexible, an Admin can assign an admin command to any user - so they can run this.

Flexible commands

1. Allow admins to give permissions to any user to run a command. /admin enable [command] [user] - The command should display a list of all admin commands. They can then select one, and the member can use it.
2. Even better, /admin enable [command] [role] - the same, but it will give all people in that role the ability to use the command.
3. If the above is not possible - /admin reset leaderboard, /admin change_winner - should be locked to Admin, and all other admin commands can be for Moderators (i.e. anyone with the "manage server" role)

Toggle on/off options

1. MVP vote

We are decreasing base mu from 8.33 to 5-7. The actual number is yet to be decided.

This will decrease the drastic differences in MMR early on. Although the default for TrueSkill is recommended it's mostly for public multiplayer games where players are meant to be playing 100s of games.

With the use case for this bot, it's unlikely individual players will reach 100s of games played. This means the algorithm may never provide an accurate rating of a player's skill as they'll never reach enough games.

We propose increasing the default confidence value so MMR gains/losses are less drastic, displaying a more "realistic" MMR with fewer games played.

The multi-op. gg link should automatically be sent to the channel like the auto draft. This should only be sent for League of Legends.
Most likely, we have no use for the op. gg command itself.

When a player is offline, it displays their Discord ID sometimes, lets's find a way always to show a name. This can look messy and unclean

Leaderboard command that uses a channel to display a top 10static leaderboard that automatically updates.

maybe /top_players - should be based on Wins > Win rate > MMR

• Some discord servers have been doing Seasons, or Splits. May be good to have an admin command that can reset the leaderboard, so they can have a fresh start. Only for that guild!

/setup [game]
A command to set up the server automatically for the admin instead of creating the channels themselves.

This will:

1. Create a category called "InHouse"
2. Create a text channel in that category called "queue" - Automatically be enabled as a queue and sends a queue straight away.
3. Create a text channel in that category called match-history - Automatically enabled as the servers match history channel. Embed saying, "Match histories will be posted in here!"
4. Create a text channel in that category called Top-10 - This will have an embed message saying
   "This channel will be used as your automatic leaderboard. Please run /admin top_ten [channelname] after you have played at least one game."

Default permissions for all these categories are that no one can talk in them. Apart from admins, obviously!
As discord does not care if two categories have the same name, this command can be run multiple times.

A way for 2 players to be able to queue and be on the same team

Could we move this to a drop-down menu underneath the match history?
Each individual in the game can interact with?

DMs need to be more consistent.

Let's include the team and role and ensure the person is not on the list, just like we do for the duo queue.

When ten players have been found, do not organise the teams into balanced teams until the game begins. Some players are dodging if they don't like the look of their team.

Let's move the duo system to this state of the queue as well.
The current duo system is a little flawed because if there is 9 people in queue, and the last person joins, they can't duo because the game automatically starts. So duos need to join early.

Lets have an error for when this happens

1. When there isn't a winner log channel set in the server
2. A game is played, and an admin runs admin change_winner
3. There should be an error that says (You do not have a winner log set, please set one with /setwinnerlog [channel])

A command that can be run after an initial set-up to make sure the bot is ready to be used

1. /verify
2. Bot attempts sends a message to the queue channel (Check! or something)
3. Bot sends a message to the winnerlog
4. Bot creates test lobby and voice channels
5. Bot deletes test lobby and voice channels

If all these pass then a success message is sent and the server owner is informed that there are no permission issues.

• Remove players who did not ready up from queue instead of removing all players

• Support for dynamic button labels
• Leaderboard for different games and an overall leaderboard
  /admin top_ten [game] [channel]
• Changes in setchannel and setwinnerlog to support game parameter

Planned Games:

• Overwatch
• Valorant

Should be released with - #230

I don't know the cause of this issue because I am never there to experience it. But at times, the bot will bug out, and players cannot queue any longer because they are stuck in a phantom game they cannot cancel. 2 issues at the same time on the 4th of December from support discord. The bot did not send winners to their winner log; they were stuck because the game was finished and not finished simultaneously.

Please read this message thread - https://discord.com/channels/1028306608100483082/1028307549608484974/1049467927470014474 (Support discord)

admin cancel does not work because that only works on current ongoing games.
Maybe a new command to fix this potential bug

New command: admin void [gameid] - will remove the game from the DB and unblock the players as a last resort

1. Let's add a warning when the admin reset leaderboard command is run. Same as we do for setting a winner log.

"This will reset all member's wins, losses, MMR and MVP votes back to 0. Are you sure?" X / ✔️

2. Since we renamed the admin reset user to user_dequeue - We can now use the admin reset user to reset a single user's leaderboard stats.

Let's also include the warning message.

"This will reset all [nickanme]'s wins, losses, MMR and MVP votes back to 0. Are you sure?" X / ✔️

• Once 10 players have readied up, the Bot should send an invite link to the created lobby
• This lets players easily find the lobby if necessary, especially in big servers with many channels.

A set-up where server owners can set up a generic game if they want to use the bot but do not play LoL, Overwatch or Valorant.

There will be a 4th option of "other" - where a server owner can set

This will make sense to set up after the bot has been released for Overwatch and Valorant, or it may be released simultaneously.

So I suspect the setchannel command will change to setchannel [game]
The options will be: LoL, Overwatch, Valorant, and Other. When they select "Other" - it should will create a generic queue "Player 1 - Player 5" and have a generic embed image (I will provide this)

Note: I have parked the need to custom lobbies of more or less than 5v5. Almost all competitive games are 5v5. If people start asking for more or less than 5v5 then we can consider.

Things to keep in mind:

1. I would like to change the Banner image per game, and I will make a generic one for "Other."
2. We may need to rewrite how the buttons work
3. /start should recognise what game the channel has been set to and then start the correct queue.
4. The Opgg embed sent in every channel now should only appear in an LoL lobby.
5. The autodraft sent in every channel now should only appear in a LoL Lobby

May release along side - #217

In the match history, adding the player's role to their name would be good. I will use an emoji for this.

:top :
:jungle:
:mid:
:adc:
:support:

Let's try reorganising the players to be shown in this order.
Can provide emoji ids.

Of course, these emojis will be different once we add support for Valorant and Overwatch so let's keep this in mind.

When multiple people click a role at the same time, because of the slight delay, the bot seems to let everyone through at once, and there is a chance of multiple roles on the same team and more than 5 players per team. Players then need to leave 1 by 1 to solve this.

Is there any way we can stop this from happening? Maybe be more strict on how it handles the queue join, 2 players only? I am not sure

Need a way to enable or disable the mmr matchmaking.
Some users want to decide on teams

/admin enable/disable MMR

Once this this is enabled, the Switch teams button returns and no MMR is counted for wins/loss. The counter also does not go up and neither does MMR.

A command that would put the entire bot into test mode, meaning they only need 2 people to test the functionality of the bot instead of 10. When we enable this manually, we edit the code and queue TOP. This command should do that.

1. /admin testmode [on/off]
2. All buttons but the first queue button are disabled/greyed out -
3. 2 people Queue Top (this name will be different depending on if its valorant or overwatch)
4. bot works as usual

We may need to discuss this as it could clash a little with #149

If it becomes too complex or hacky, let us scrap it and look for a dev-only command for me only.

We need a way to back up the SQLite DB from InHouseQueue. Automated maybe and then stored on s3

Currently, all queries are hard-coded and difficult to maintain as a result. I propose we add a database ORM like SQLAlchemy implemented similarly to https://github.com/mrtolkien/inhouse_bot to manage the database and queries.

• Create a docker container for the bot

Will make it much easier to deploy, especially for other people

Hard to know who is who sometimes, so this will make it clearer

• Let's add the role the person played to the MVP vote DM

The top 10 stops auto-updating after a while. You need to rerun the command constantly, and it will update.

I have tried reproducing it, but it seems it doesn't work now.
Another solution would be to resend the updated leaderboard and delete the previous one. The same way we do with the queue, deleting it as soon as the winner log is posted.

A system where 2 players can queue up and be on the same team 100% of the time.

Potential solution:

1. Both players join the queue as normal.
2. There will be an extra new button on the queue called "Duo". (Should only work when MMR is enabled)
3. One player can click this, and a drop-down should appear. The drop-down will have the names of all players in the current queue. They select the player they want to duo with.
4. Once a player has been selected, the other player receives a DM about the duo request. The player waiting also receives a ephemeral message that a DM has been sent.
5. They accept/reject in the DM
6. On acceptance, both players will be on the same team regardless of the MMR difference.
7. Once acceptance, one player will physically move to Red/Blue side to join their duo and an emoji is next to their name showing they are duo.
8. If the player cant move to the other side because its already taken by someone else, then an error message should appear. "This role on red/blue side is taken". We should not priortise duos so much that 2 can skip the queue over someone else.
9. On reject, nothing happens, and they both stay in the queue, and MMR decides on their team.

Things to keep in mind:

1. There should be some visual for others to know two players are a duo - maybe a star next to both names.
2. If for whatever reason, the person's DM is private and the bot cannot send a DM, it should also send an ephemeral message stating this.
3. Duo system should be toggle on/off per server.
4. Make sure both users are of different roles - E.G both users can both queue Top, and then be allowed to click duo.

• Adding Losses to leaderboards
• `` code block generates a cleaner look and feel.
• Use emojis for top 10 leaderboards 1-10.
• We can find out what role the person plays the most and use an to put next to their name. This can be done with emojis

Examples:

I see a lot of people typing in the queue channel.

I think we either need an extra step that warns people that they will unable to type once this is set.
Example
/setchannel #channelname

Embed = "Message in this channel will automatically be deleted to keep the queue channel clean, do you want to proceed?"
Then a button to say yes/no

A command to set queue preference;

• Players can only be in 1 queue at a time
• Players can join multiple queues but will be removed from others when one of their queues reaches the ready up phase

Command to display the member's rank in the server

/start normal - Will create a normal queue, players can switch teams
/start mmr - Will create an MMR queue, teams allocated according to mmr

/start normal
Players will not gain MMR who play in this queue, but will gain a win on the leaderboard

Players need a way to create a Multi op.gg link so they can see all players on the enemy team on one page

This will only work if players have their IGN as their discord nickname. So that is down to the server to enforce

Example op.gg link: https://www.op.gg/multisearch/euw?summoners=iHenners%2CAEBE%2CBolmanJr%2CTheForbiddenKing%2Cx+Blue+Eagles+x%2CTemodjin

So the command, or button or however we decide to do it, will take the Red and Blue team Nicknames and convert this into a link like the above.

Spaces between players have this %2C
Spaces between the player's IGN, like The John - will work. So we don't need to worry about handling that.

In many servers players, nicknames are prefixed with IGN:. For example, my discord nickname would be IGN: iHenners. We only want the iHenners part of the name. So we need to filter out stuff like IGN:, ign, ign:, IGN at the start of a nickname. If there isn't IGN, it should just take the whole nickname. This is where Regex might help.

So the command or button would look something like this in the code

f'https://www.op.gg/multisearch/[region]?summoners={nickname1}%2C{nickname1}%2C{nickname1}%2C{nickname1}%2C{nickname1}%2C

Another thing to note, they should be able to set the region https://www.op.gg/multisearch/[REGION]?summoners=

Valid regions - euw, na, eune, oce, kr, jp, br, las, lan, ru, tr

I suspect if it is a command, it will look something like

/opgg [region] [gameId] [red/blue] and this will send the op.gg link as a message.
It would be even more impressive if it could only be sent in that lobby channel. But not a priority.

Example question asked on StackOverflow: https://stackoverflow.com/questions/70320440/discord-py-bot-make-op-gg

1. Once a game has ended, the queue embed should be deleted from the queue channel. There is no need to keep it in there. Every info someone needs should be in the match history channel. It's crucial that it only gets deleted once the game has ended and there is a winner log.

2. There is a problem with the bot: when the server restarts, the last queue before the restart is buggy. Solution - whenever the bot restarts, it should automatically send a new fresh queue in the queue channel.

If someone doesn't ready up, they are kicked from the queue after 10 minutes. However, it looks like everyone is kicked out. The person who did not ready up is only removed "visually" on the queue embed. On the backend, everyone is kicked. Not working as intended currently.

Workaround: Everyone must re-queue.

Solution
Given someone does not queue up in 10 minutes
When they are removed from the queue
Then ONLY that person is removed from the queue
And the queue goes back to queue state, from ready

If someone does not vote for 1 game and they played another game, their following vote number will count twice?
Or at least the bot responds with 2 "Thank you for voting" messages

Servers who have sbmm disabled will not have a /leaderboard result as this command defaults to mmr.

Players can select which champion they played. This can be a dropdown list and be part of the match history

Once it's set, it can't be changed.

Once they select which champion they played, this can be added to the member_history table as an extra tracker.
The champion they choose will show as an emoji next to the name in the match history (it will update the embed as they pick).

The champion should now also be an option on the /leaderboard command.

/leaderboard [character] - Will show players with the highest win rate on that champion/agent/hero. Let's limit this top 5. The thumbnail can also be the champion character.

/rank mvp/mmr - should display the champion they have played the most instead of the author ID

League of Legends = champion
Valorant = agent
Overwatch = hero 

Maybe we can use this naming convention depending on what game that would be a nice touch, otherwise we can just stick to [character] for everything.

League of Legends
One way to dynamically get the champion images:
https://ddragon.leagueoflegends.com/cdn/13.1.1/img/champion/{champion}.png

The first character in the champion's name is always uppercase. I can provide the name of all champions....
Example: https://ddragon.leagueoflegends.com/cdn/13.1.1/img/champion/Kaisa.png

We can grab the latest version of the game here -
https://ddragon.leagueoflegends.com/api/versions.json

And use that as well. New champions are added every 3 months or so. So It may look like
https://ddragon.leagueoflegends.com/cdn/{version}/img/champion/{champion}.png

Grabbing the latest version is beneficial because when they add a new champion, we will only need to add the champion's name to the list instead of editing the game version. Also, some champions get aesthetic reworks.

For Valorant:
https://valorant-api.com/v1/agents

So for the agent display icon, the path would be
['data']['displayname'] to get the agent name

then
['data']['displayIcon'] for the image

Overwatch
https://overfast-api.tekrop.fr/#tag/Heroes/operation/get_hero_heroes__hero_key__get

https://overfast-api.tekrop.fr/heroes/{hero_key}

Example:
https://overfast-api.tekrop.fr/heroes/cassidy
And we care for ['name'] and ['portrait']

If for whatever reason a champion doesn't exit the icon should be ❓
I can get you all the names of all characters in all the games when needed. But maybe with these api calls we wont need that.

Had suggestions from 2 different people to allow people to switch sides.

• Be able to switch from blue > to red and vice versa as long as there is an available slot on the other side.

This way people can make teams a little easier / quicker

Create the draft using https://draftlol.dawe.gg/
Send all 3 links into the channel. This doesn't need to be a command but should be sent as soon as the channel is made.

Once we add support for more games, it will only be sent for LoL

MVP vote dm - /admin MVP [enable/disable]
OP.GG Links /admin opgg [enable/disable]
LOL Draft Link /admin loldraft [enable/disable]

Customize the ready check timer /ready [mins]
voice channels will not create any voice channels, but will always still create lobby channel

Servers can opt out of getting the message in the queue whenever the bot restarts

Update how many members to show in "top_ten". Maybe a server wants top 3, or top 5.
I suspect having 3,5,10,15,20. As options would be a good start

For busy servers, people have noticed that it's hard to find what game to spectate, especially if there are a lot of queues happening.

• let's add the spectate buttons to the lobby channel as well as the queue channel.

Adding a Discord status to the bot

E.g

playing !help 

1. Game ends, and 1 team is accidently declared the winner - all players receive/lose MMR
2. Admin runs /admin change_winner [gameid] Red
3. Wins are converted as expected, but MMR is not. MMR still remains the same

Seems like an oversight on our part. Luckily this doesn't happen often but would be good to look into

For Overwatch, I will add the emojis of the roles as we did for LoL, as it works precisely the same.

Overwatch roles:
Tank
DPS
DPS
Support
Support

Valorant Roles:
Controller
Initiator
Sentinel
Duelist
Flex - Flex is not a role, but it can be any of the above roles, depending on what is needed. When this is selected, allow the person to select the role they will play via a drop down.