Implement AMA bot from https://github.com/NoComment1105/cozy-ama #71
Conversation
module-ama/src/main/kotlin/org/quiltmc/community/cozy/modules/ama/AmaExtension.kt
Outdated
Show resolved
Hide resolved
module-ama/src/main/kotlin/org/quiltmc/community/cozy/modules/ama/_Utils.kt
Outdated
Show resolved
Hide resolved
|
I was able to test and have since done so and fixed a couple of issues i found. All is ready for review |
module-ama/src/main/kotlin/org/quiltmc/community/cozy/modules/ama/_Utils.kt
Outdated
Show resolved
Hide resolved
| import dev.kord.common.entity.Snowflake | ||
|
|
||
| public class MemoryAmaData : AmaData { | ||
| private val ama: MutableList<AmaConfig> = mutableListOf() |
There was a problem hiding this comment.
Nitpick: might it be reasonable to instead use a map here?
There was a problem hiding this comment.
Since configs are only referenced by guild ID, it could give a slight speedup. It also fixes a potential bug: currently any changes caused by running setConfig will be entirely ignored as only the first matching item is chosen
There was a problem hiding this comment.
I think it would be a guildId to AmaConfig map
|
Just talked about this in AMA chat but this currently has no support for plural kit. It sounds like it would not be feasible to add and have it work properly however should an optional name field be added to the ask modal so that people part of plural systems can have their name shown in the question. Here's a little bit of a bad mockup: |
|
Ladysnake's fork of Cozy does this for suggestion submissions:
A similar feature could be made for the AMA modal |
# Conflicts: # .idea/modules.xml # src/main/kotlin/org/quiltmc/community/database/migrations/v22.kt
|
|
||
| if (config == null) { | ||
| respond { | ||
| content = "There is no AMA config for this guild!" |
There was a problem hiding this comment.
Maybe print the command to set the config?
|
|
||
| check { | ||
| anyGuild() | ||
| hasPermission(Permission.ManageGuild) |
There was a problem hiding this comment.
Does this allow event team members to start the AMA?
| import dev.kord.common.entity.Snowflake | ||
|
|
||
| public class MemoryAmaData : AmaData { | ||
| private val ama: MutableList<AmaConfig> = mutableListOf() |
There was a problem hiding this comment.
I think it would be a guildId to AmaConfig map
Untested locally due to discord not updating my bots commands, hopefully someone else can see if it works?
EDIT: It was rate limiting please holdEDIT 2: Nevermind it just won't start plz help