shibdib / Dramiel

EVE Discord Bot
MIT License
42 stars 26 forks source link

To Do List (Requests go here, bugs/issues DO NOT) #74

Closed shibdib closed 2 years ago

shibdib commented 8 years ago

Current Jobs (Released, in testing)

URGENT Nothing ATM

Medium Priority

Low Priority

Completed and passed testing

Scrapped

rufjame commented 8 years ago

Any chance of having an "allow all but this channel" for the bot to respond?

shibdib commented 8 years ago

sure, added to the list

shibdib commented 7 years ago

@rufjame added https://github.com/shibdib/Dramiel/commit/e419d769df7ccf654f49e7fe1b56aa79c6661515

CokkocZateki commented 7 years ago

79 works well. so far no problems.

freeziepopz commented 7 years ago

Any chance in getting kills with supers/caps on it to ping the channel and give the jump route from pre configured "Home" System, or atleast Light years away from pre configured system? No just kills your corp/alliance are on, just kms in general

shibdib commented 7 years ago

Not likely. That's a lot of code for a very niche feature.

On Nov 11, 2016 9:06 PM, "freeziepopz" notifications@github.com wrote:

Any chance in getting kills with supers/caps on it to ping the channel and give the jump route from pre configured "Home" System, or atleast Light years away from pre configured system? No just kills your corp/alliance are on, just kms in general

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/shibdib/Dramiel/issues/74#issuecomment-259997469, or mute the thread https://github.com/notifications/unsubscribe-auth/AFguZQmoKS_pt9j4wowvaNC91RHYaB8Tks5q9JlxgaJpZM4KdAEk .

shibdib commented 7 years ago

NEW Current Jobs (Released, in testing)

Siphon detection overhaul (multi corp support, and no longer matters if you level out silos when you empty them)

Let me know if anyone encounters any issues

shibdib commented 7 years ago

NEW Current Jobs (Released, in testing)

Expand auth to support multiple groups - #62

rufjame commented 7 years ago

With the 'eventual' removal of Pos' are there any plans in the pipe line for Citadel and Engineering Complex fuel notifications?

shibdib commented 7 years ago

Citadel ones should exist already. I'll have to double check but I believe I added the notification already

On Dec 12, 2016 10:42 AM, "rufjame" notifications@github.com wrote:

With the 'eventual' removal of Pos' are there any plans in the pipe line for Citadel and Engineering Complex fuel notifications?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/shibdib/Dramiel/issues/74#issuecomment-266350873, or mute the thread https://github.com/notifications/unsubscribe-auth/AFguZcKs47hxGXWGwUrfGhjG9l6Tq1A_ks5rHOXGgaJpZM4KdAEk .

rufjame commented 7 years ago

There's an eve notification for when services shutdown but I haven't seen one for the 24hour mark.

shibdib commented 7 years ago

I'll do some digging. Does one pop up in game? And if so does the bot post it saying to make a github issue with this info etc.. Etc..

On Dec 12, 2016 10:48 AM, "rufjame" notifications@github.com wrote:

There's an eve notification for when services shutdown but I haven't seen one for the 24hour mark.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/shibdib/Dramiel/issues/74#issuecomment-266351590, or mute the thread https://github.com/notifications/unsubscribe-auth/AFguZR6eBBn_lSVnkEtgUf7WYwOEpEQ7ks5rHOcpgaJpZM4KdAEk .

rufjame commented 7 years ago

Engineering Complexes give a notification when the services shutdown and that's it lol. It's possible to see the fuel hangar from the corporate asset API though.

On Mon, Dec 12, 2016 at 6:08 PM, shibdib notifications@github.com wrote:

I'll do some digging. Does one pop up in game? And if so does the bot post it saying to make a github issue with this info etc.. Etc..

On Dec 12, 2016 10:48 AM, "rufjame" notifications@github.com wrote:

There's an eve notification for when services shutdown but I haven't seen one for the 24hour mark.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/shibdib/Dramiel/issues/74#issuecomment-266351590, or mute the thread https://github.com/notifications/unsubscribe-auth/AFguZR6eBBn_ lSVnkEtgUf7WYwOEpEQ7ks5rHOcpgaJpZM4KdAEk

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/shibdib/Dramiel/issues/74#issuecomment-266357698, or mute the thread https://github.com/notifications/unsubscribe-auth/AF9_LjxpbzNPILVm77nW_s4MSrZg-Gbkks5rHPMBgaJpZM4KdAEk .

shibdib commented 7 years ago

That's annoying but can make that work.

On Dec 12, 2016 11:44 AM, "rufjame" notifications@github.com wrote:

Engineering Complexes give a notification when the services shutdown and that's it lol. It's possible to see the fuel hangar from the corporate asset API though.

On Mon, Dec 12, 2016 at 6:08 PM, shibdib notifications@github.com wrote:

I'll do some digging. Does one pop up in game? And if so does the bot post it saying to make a github issue with this info etc.. Etc..

On Dec 12, 2016 10:48 AM, "rufjame" notifications@github.com wrote:

There's an eve notification for when services shutdown but I haven't seen one for the 24hour mark.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/shibdib/Dramiel/issues/74#issuecomment-266351590, or mute the thread https://github.com/notifications/unsubscribe-auth/AFguZR6eBBn_ lSVnkEtgUf7WYwOEpEQ7ks5rHOcpgaJpZM4KdAEk

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/shibdib/Dramiel/issues/74#issuecomment-266357698, or mute the thread https://github.com/notifications/unsubscribe-auth/AF9_LjxpbzNPILVm77nW_ s4MSrZg-Gbkks5rHPMBgaJpZM4KdAEk .

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/shibdib/Dramiel/issues/74#issuecomment-266358348, or mute the thread https://github.com/notifications/unsubscribe-auth/AFguZbpBHBKCGZU7MqEKD3ZqWAH6tLUzks5rHPRAgaJpZM4KdAEk .

ZimmerA commented 7 years ago

Hi, can u add support for multiple roles per group? I fiddled something together but i think it would be nice to have it officially supported.

I'll put my code down below.

if ($corpID === $authGroup['corpID']) {
    foreach ($roles as $role) {
        foreach($authGroup['corpMemberRole'] as $discordRole){
        if($role->name == $discordRole){
                $member->addRole($role);
            $corpRoleSet = 1;
        }
    }
    }
}

This should replace line 184-191 in auth.php(note that alliance also has to be done if you want multiple alliance roles). Obviously you have to change the corpMemberRole in config.php to be an array of strings.

This works as far as that the bot sets multiple roles in discord, however i have no idea if this affects any of the other plugins since it was just a quick solution.

Greetings

shibdib commented 7 years ago

when would something like that be used? If everyone gets the same 4 roles it's no different permissions wise than if everyone got 1 role

ZimmerA commented 7 years ago

You're right... I was thinking the wrong way because of the way our CEO setup Discord before we used your bot.

In case you're interested: Our corp had the roles: "Council", "Member", "CEO" and "Corp". When we manually did them we added the members to "Corp" and "Member" the CEO to "CEO" and "Corp" and so on. This would allow us to do @corp and reach every corp member of every group. What we could have just done is add the CEO and council to "Member" too. I didn't think of that because our member role has a very specific name.

Thanks for clearing this up.

shibdib commented 7 years ago

Yup was confused, although just gave me the idea of auto giving out ceo roles :P

nizzan commented 7 years ago

ToDo/Tickets The ability to: !nticket Task 1 | Someone needs to do x -> this creates a New ticket with Task 1 being the title and the text after being the description. !atickte Task 1 | Me -> This assigns the task to me !cticket Task 1 | Done -> this closes the ticket marking it with a solution of "Done" !lticket -> list tickets

Thoughts?

shibdib commented 7 years ago

That's starting to get outside of the scope I was aiming for in making an eve bot. Not sure how a ticket system is eve related

CokkocZateki commented 7 years ago

Can i request that the SSO function also works so that Alliance/Corp Blues can use the service. Currently we have to add the corp to the config file.

@shibdib

Aarondorn2 commented 7 years ago

Another feature request:

As a bot admin, I would like to configure the mail plugin to post personal mails to a given character to a channel ID, so that I can have an in-game-character that represents the bot.

We have a character with the same name as the bot in-game and would like to be able to send personal mails to it and have those posted to a channel - not just mails from the corp/alliance.

o7

shibdib commented 7 years ago

Mmmm I can probably do something along the lines of, every mail a certain API Key gets is posted.

Aarondorn2 commented 7 years ago

That would work. It's kinda cool having the bot tie in to the game with an actual character - but Im not sure if that's something that would be widely used among other users. So maybe a low priority?

CokkocZateki commented 7 years ago

well with Alpha clones it could become more popular to do something like that

shibdib commented 7 years ago

low priority but super easy to implement. Basically just turning off the filter.

BroodLK commented 7 years ago

If using both Corp and Alliance auth settings and someone leaves alliance and re-auths with corp. It does not remove the alliance role. It does however properly add the corp role.

rufjame commented 7 years ago

very low priority but any chance of blacklisting users from being able to use the bot? Namely other bots that /tableflip whenever someone unflips it. Makes for an interesting channel for a while....

nizzan commented 7 years ago

Multi-Corp support for fuel notifications. We have a few alt-corps we'd like to see the fuel notifications of in the same channel as our main corp.

IronWarjack commented 7 years ago

Didn't see this listed. The bot responds with the info back to the current channel publicly. And, while not a horrible thing, we suddenly find the channel bogged with everyone doing time checks, char, tq, etc. We use to use a slack version that did the same things, but it returned as a private message to the user. We'd love a flag that can turn this feature on (for those who want it).