Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Got logging of all users in group working #59

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

MrShoes725
Copy link

No description provided.

@gregzaal
Copy link
Owner

Can you explain what this does?

Looks like the help info was copy pasted and not filled in.

Please use PEP8 standard.

@Jakimbo
Copy link

Jakimbo commented Aug 13, 2020

Sorry about that, this is a WIP and MrShoes is still learning his way around git, he didnt mean to send in a pull request

@Jakimbo
Copy link

Jakimbo commented Aug 16, 2020

Ok, I've gotten it working to a point where it is functional and clean, but I still have some ideas ill be working on.

The features I've added were made to solve a problem we were having playing Sea of Thieves on alliance servers with multiple ships (don't judge). Our problem was, when having a lot of people (sometimes up to 10 people) in the same voice channel it became chaos, especially during times of heavy action like PVP. When that happened we had to have a discord admin sort everyone out into separate channels by ship manually, and then move everyone back into the main channel when things cooled down. This was tedious and confusing since one person had to remember where everyone was, and this would leave us down 1 player and interrupt comms while people were being moved around.

So we came up with the idea for a bot that would allow us to make a voice channel for each ship, and then with a couple commands pull everyone back into one main channel, remember where they all were, and split us back apart instantly when needed.

So I've added 4 commands to this bot.

  • creategroup (also cg): This command will create a category, 2 voice, and 1 text channel like so.

image

The "new session" channel is the same as before, and the text and "Tavern" channel don't do anything special on their own.

  • merge (also m): Once users have used the "new session" channel to split into groups, you can use this command to move everyone back into the "Tavern channel"

*Note: Once the merge command is run, the auto channels in that category will no longer be deleted when they are empty. To allow them to be deleted again the resetgroup command must be run

Before merge
image

After merge
image

  • split (or sp): Once the merge command is run, this will move everyone back into the channels they were in before being placed in the "Tavern"

Before Split
image

After Split
image

  • resetgroup (or rg): This needs to be run once the gaming session is finished. It clears out the groups' save file so that the auto channels in that category will no longer be prevented from being deleted.

*Another Note, if you run your own bot using this fork, you will also need to run mkdir groups so it can store the group settings
*Last note: I did change the functionality of the create_secondary function to always force a new secondary channel to the top of the list. That was for my own personal preference

Let me know what you think and if you have any ideas to improve the functionality or the code itself. This is my first time working with python so I can guarantee there are better ways of doing some of this, thanks!

@vinanrra
Copy link
Contributor

Nice idea, rly good for clans

@Mixarar
Copy link

Mixarar commented Oct 23, 2020

I can't run bot I get error in functions.py. I am running Debian Raspbian buster

@Mixarar
Copy link

Mixarar commented Oct 29, 2020

I fixed it but now bot doesn't create channels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants