valkyrjaproject / valkyrja.discord Goto Github PK

Implemented in Server where every config reload would compile a list of strings into a list of regexes to save that performance.

This should be an easy fix of editing the Utils timestamp functions to also accept an extra parameter - to be an integer UTC +-number, and use globalconfig const bool IsSummerTime.

• ServerConfig value + webconfig

https://i.gyazo.com/bd851d3c4c3cfd30aff6656979ce37f6.png
a specific option for mention spam to block separate instances of mentions as anti mention spam
as shows in the picture above

Post to the channel used for User Join/Leave saying when a user has verified themselves using code verification

This project seems interesting. But to learn more, it points to its website in multiple instances. Nothing wrong with that except that the website has been down for days.

• Random responses
• Optional argument {mentioned} - this would have different response if the message does not mention anyone.

!promote @user @ROLE, @ROLE, @ROLE

Would be more convenient to declare the roles by handle and providing a comma separated list of roles.

TL;DR: publicRoles shows every role prefixed with a pound sign, but join only lets you join if you drop the pound sign. This is pretty confusing at first glance.

I guess the fix would be to either drop the pound sign from publicRoles or have join interpret roles with the pound sign?

Hey there! I just thanked someone and realised I turned ping reply off and it didn't thank

I feel like that's an oversight, and I looked at the code, imho it'd be fairly easy to implement additionaly to the current mention system, by adding it to the end. Also if I'm not mistaken, you don't check whether someone has already been thanked in a message?

For example, would @LukeChriswalker @LukeChriswalker @LukeChriswalker @LukeChriswalker @LukeChriswalker add five Karma?

Regardless, I wanted to first open this issue to gauge, whether there is interest to implement the non-pinging thanking, cuz I don't wanna implement something and then the maintainers don't like it lol

Just ping me if it's non-intentional, and I'll go and implement it 👍🏻

Best, Lena

P.S.: A screenshot of the thing would be nice, the first one will thank, the second one won't

The title of this issue might be a little crude.

TLDR; Commands are only detected when lowercase, but documentation displays them containing uppercase letters.

The problem I fell into:
I read the documentation (https://valkyrja.app/docs#silentban) and thus assumed as the command is described as silentBan that I need to execute the !silentBan command in Discord.
Result was that the bot did nothing, it just ignored my command.
After a week now I found out that the command only works when I write !silentban because the check (https://github.com/RheaAyase/Valkyrja.core/blob/master/Botwinder-client.cs#L750) is case-sensitive and the command (https://github.com/RheaAyase/Valkyrja.discord/blob/master/Modules/Moderation.cs#L750) is defined as lowercase.

I wanted to just make a Valkyrja.core pull request to fix this but I came up with multiple solutions:

• make commands always handled lowercase internally, but that would break non lowercase Custom Commands.
• make the Dictionary case insensitive by passing a StringComparer.InvariantCultureIgnoreCase to the constructor (https://github.com/RheaAyase/Valkyrja.core/blob/master/Botwinder-client.cs#L63)
• fix the documentation and just write silentban in lowercase (https://github.com/RheaAyase/Valkyrja.web/blob/master/www/resources/views/features.blade.php#L645)

Which solution do you think is the best? I'll create a pull request for that.

Research needed to get a list of the most popular ones.

If you want to discuss with me, do it. But dont just make an argument and block me afterwards. Either feed the trolls or leave them.

• Implement localisation system.
• Custom language support
• Web editor
• Move all the original strings into English
• French
• Italian
• German
• Spanish
  EFIGS would be the must-have for any localisation, but we can have more...

Nvm, already exists I'm dumb :)

Design help wanted =]

• The idea is that people get experience points for messages sent, for images sent, etc...
• Option to give Karma cookies for levelups.

PublicRoles upgrade - You will be able to create groups of public roles, where the user will have only one of the roles at a time. If they !join another, the previous one will be removed. For example the Elite Dangerous server has public roles for allegiance and factions. These would be two groups, where you can be only either imperial or federal, and you can be part of only one of the factions.

Fix is to create an exception for messages that would trigger as duplicates, but are not exact match, and begin with http or <http...

• should handle it correctly and spit out message about having too many parameters... or just repeat the same one for other syntax issues

The bot will PM subscribed people with latest patchnotes when they are released.

• Implement !subscribe command with appropriate storage...
• Implement !sendPatchnotes owner command to execute it.

It may require modifying the base library though, in order to group requests together once per second to avoid ratelimits (modifying channel with people who can or can not talk at the time)

Minor annoyance.

So, it seems like if you ban a user and then try to run %clear against them, it will not work.

step 1: ban someone without clearing their messages
step 2: wait a few minutes
step 3: try clearing their messages with %clear