aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | | | | | * | | | | | | | Do not add suggestion for tags with short names if a group is specifiedGravatar Numerlor2021-07-06-6/+14
| | | | | | | | | | | | | |
| | | | | | * | | | | | | | Update outdated docstringGravatar Numerlor2021-07-06-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The saving functionality has not been present on the bot for a while
| | | | | | * | | | | | | | Fix testsGravatar Numerlor2021-07-06-10/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unnecessary invoked with mocks were removed and some more checks added for the new behaviour
| | | | | | * | | | | | | | Add leading ยป when listing tag suggestionsGravatar Numerlor2021-07-06-1/+1
| | | | | | | | | | | | | |
| | | | | | * | | | | | | | Remove unnecessary typehintGravatar Numerlor2021-07-06-2/+2
| | | | | | | | | | | | | |
| | | | | | * | | | | | | | Update strings to use double quotesGravatar Numerlor2021-07-06-9/+9
| | | | | | | | | | | | | |
| | | | | | * | | | | | | | remove unused thresholds parameterGravatar Numerlor2021-07-06-8/+2
| | | | | | | | | | | | | |
| | | | | | * | | | | | | | Move tag search to new designGravatar Numerlor2021-07-06-13/+20
| | | | | | | | | | | | | |
| | | | | | * | | | | | | | Add option to list all tags in a groupGravatar Numerlor2021-07-06-2/+19
| | | | | | | | | | | | | |
| | | | | | * | | | | | | | Move tag listing to new design and move it outside of tag display methodGravatar Numerlor2021-07-06-73/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The display method was renamed to get_tag_embed and now exclusively handles embed for a tag/suggestions instead of holding the logic of the whole command fixup! Move tag listing to new design and move it outside of tag display method
| | | | | | * | | | | | | | Move cooldown handling to the Tag classGravatar Numerlor2021-07-01-34/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of the Cog keeping track of cooldowns of all tags, every tag now handles its own cooldowns which are registered with the `set_cooldown_for` method. This change also fixes the bug where cooldowns can only be on cooldown in only one channel at a time, with invokations in other places cancelling cooldowns.
| | | | | | * | | | | | | | Base functionality of tag fetching with groups and in file metadataGravatar Numerlor2021-07-01-89/+167
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The code was restructured to hold tags and their identifiers in individual classes and some methods moved to function to detach some of the not directly related functionality from the cog class
| | | | | | | | | * | | | | Skip private channels when deleting from allGravatar mbaruh2021-12-06-5/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When specifying all channels, the command now skips private channels to optimize for speed.
| | | | | | | | | * | | | | Properly check the channel when deleting from cacheGravatar mbaruh2021-12-05-14/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously the cache was only used to delete from all channels. I didn't add a channels check when I changed it.
| | | | | | | | | * | | | | Remove now redundant input check.Gravatar mbaruh2021-12-03-4/+0
| | | | | | | | | | | | | |
| | | | | | | | | * | | | | Update _build_predicate to require a limitGravatar mbaruh2021-12-02-10/+8
| | | | | | | | | | | | | |
| | | | | | | | | * | | | | Simplify cache usageGravatar mbaruh2021-12-02-39/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removes the cache usage argument from the clean commands. Cache usage is now an implementation detail. The cache will be used if the age of the oldest message requested for cleaning is younger than the oldest message in the cache. Additionally fixes the logger to the one used in the rest of the bot (caused by a faulty merge).
| | | | | | | | | * | | | | Remove cleaning based on number of messagesGravatar mbaruh2021-12-02-49/+44
| |_|_|_|_|_|_|_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All clean commands now use the clean limit (message, time delta, ISO datetime) instead of `traverse`. Consequently, `clean all` has been removed as `clean until` now effectively fulfills that role.
* | | | | | | | | | | | | Dynamic views for command help embeds (#1939)Gravatar Boris Muratov2021-12-01-51/+141
|/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Dynamic views for command help embeds Adds views for commands to navigate groups. For subcommands, a button is added to show the parent's help embed. For groups, buttons are added for each subcommand to show their help embeds. The views are not generated when help is invoked in the context of an error.
* | | | | | | | | | | | Merge pull request #1868 from python-discord/subscribe-with-buttonsGravatar ChrisJL2021-12-01-88/+248
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Subscribe with buttons
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'main' into subscribe-with-buttonsGravatar ChrisJL2021-12-01-26/+53
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
* | | | | | | | | | | | | Merge pull request #1928 from python-discord/kill-sir-threadevereGravatar Matteo Bertucci2021-12-01-26/+53
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Manage nomination threads in bot
| * | | | | | | | | | | | | Remove redundant Union in a type hintGravatar Chris Lovering2021-12-01-1/+1
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | Use presence of a nominee as check for pending reviewsGravatar Chris Lovering2021-12-01-1/+1
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | Always return a review string for a given nominationGravatar Chris Lovering2021-12-01-8/+5
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | Fix emoji reaction error in reviewerGravatar Chris Lovering2021-12-01-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using a :eyes: style emoji string in a ctx.add_reaciton call will error. Discord expects either a unicode emoji, or a custom emoji.
| * | | | | | | | | | | | | Ensure a nomination archival emoji isn't from the botGravatar Chris Lovering2021-12-01-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is most relevant in local dev testing where the Emojis.check_mark could be the same as the Emojis.incident_actioned or Emojis.incident_unactioned, which would cause the bot to attempt to archive the post_review invocation if it was posted in the nomination voting channel.
| * | | | | | | | | | | | | Supress NotFound when archiving a nominationGravatar Chris Lovering2021-12-01-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This supresses both the mesage deleteions and the thread archive, so that if they are removed before the code can get to them, it does not raise an error.
| * | | | | | | | | | | | | Update nomination message regexGravatar Chris Lovering2021-12-01-2/+1
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | Manage nomination threadsGravatar Chris Lovering2021-12-01-1/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change creates a thread while posting the nomination, and then archives it once the nomination is concluded.
| * | | | | | | | | | | | | Refactor make_review to return nominee tooGravatar Chris Lovering2021-12-01-9/+9
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | Add thread archive time enum to constantsGravatar Chris Lovering2021-12-01-0/+12
|/ / / / / / / / / / / / /
| * | | | | | | | | | | | Subscribe command replies to invocation to keep contextGravatar Chris Lovering2021-11-30-1/+1
| | | | | | | | | | | | |
| * | | | | | | | | | | | Add lock emoji to highlight unavailable self-assignable rolesGravatar Chris Lovering2021-11-30-1/+1
| | | | | | | | | | | | |
| * | | | | | | | | | | | Swap remove and unavailable colours for subscribe commandGravatar Chris Lovering2021-11-30-2/+2
| | | | | | | | | | | | |
| * | | | | | | | | | | | Sort unavailable self-assignable roles to the end of the listGravatar Chris Lovering2021-11-30-0/+3
| | | | | | | | | | | | |
| * | | | | | | | | | | | Allow roles to be assignable over multiple monthsGravatar Chris Lovering2021-11-30-23/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This includes a refactor to use a dataclass for clearer implementation. Along with that, this changes the roles so that they're always available, but un-assignable roles are in red and give a different error.
| * | | | | | | | | | | | Delete the subscribe message after 5 minutesGravatar Chris Lovering2021-11-30-1/+6
| | | | | | | | | | | | |
| * | | | | | | | | | | | Use new get_logger helper utilGravatar Chris Lovering2021-11-30-3/+2
| | | | | | | | | | | | |
| * | | | | | | | | | | | Stop listening for events when message is deletedGravatar Chris Lovering2021-11-30-0/+2
| | | | | | | | | | | | |
| * | | | | | | | | | | | Add 10s member cooldown to subscribe commandGravatar Chris Lovering2021-11-30-0/+1
| | | | | | | | | | | | |
| * | | | | | | | | | | | Ensure the user interacting is still in guild before changing rolesGravatar Chris Lovering2021-11-30-0/+4
| | | | | | | | | | | | |
| * | | | | | | | | | | | Add an interactive subscribe commandGravatar Chris Lovering2021-11-30-0/+139
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This command gives the users a set of buttons to click to add or remove pre-determined announcement roles. Adding or removing a role updates the button state to reflect the change and what would happen if the user clicks the button again.
| * | | | | | | | | | | | Add self assignable roles to configGravatar Chris Lovering2021-11-30-0/+10
| | | | | | | | | | | | |
| * | | | | | | | | | | | Remove the subscribe command from the verification cogGravatar Chris Lovering2021-11-30-67/+4
| | | | | | | | | | | | |
| * | | | | | | | | | | | Move handle_role_change to a util fileGravatar Chris Lovering2021-11-30-21/+32
|/ / / / / / / / / / / /
* | | | | | | | | | | | Merge pull request #1981 from python-discord/bsoyka/rules-shortening-spaceGravatar Johannes Christ2021-11-29-3/+3
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Small changes to invalid rule indices message shortening
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'main' into bsoyka/rules-shortening-spaceGravatar ChrisJL2021-11-29-1/+1
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
* | | | | | | | | | | | | Use bright_green for "Currently Helping" DMs (#1979)Gravatar Ben Soyka2021-11-28-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Xithrius <[email protected]>
| * | | | | | | | | | | | Refactor long lineGravatar Ben Soyka2021-11-27-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Doing this similar to how the docs command works for shortening