aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Clarifies Constants Use in SilenceGravatar Hassan Abouelela2020-11-28-23/+25
| | | | | | | Changes all usages of bot.constant to use dotted path to remove confusion and namespace collision. Signed-off-by: Hassan Abouelela <[email protected]>
* Updates Silence LockGravatar Hassan Abouelela2020-11-26-3/+11
| | | | | | | Modifies the lock on the silence command, in order to choose between ctx and channel arg based on input. Signed-off-by: Hassan Abouelela <[email protected]>
* Reduces Redundant Code in Silence CogGravatar Hassan Abouelela2020-11-25-32/+25
| | | | | | | Restructures some code to make it more understandable and reduce duplication. Signed-off-by: Hassan Abouelela <[email protected]>
* Refactors Send Message Function Gravatar Hassan Abouelela2020-11-25-11/+5
| | | | | | Refactors the send message utility function to make it more legible and reduce unnecessary calls. Co-authored-by: Mark <[email protected]>
* Reduces IsInstance Calls Where PossibleGravatar Hassan Abouelela2020-11-24-6/+12
| | | | | | | Reduces redundant calls to isinstance by saving the result where applicable. Signed-off-by: Hassan Abouelela <[email protected]>
* Reduces Redundancy in UnmuteGravatar Hassan Abouelela2020-11-24-10/+10
| | | | | | Replaces a repeated hardcoded message with a dynamically built one. Signed-off-by: Hassan Abouelela <[email protected]>
* Adds Connect to Reset PermissionsGravatar Hassan Abouelela2020-11-24-2/+2
| | | | | | | | During unsilencing, if the previous channel overwrites are None, the channel should default to None for all relevant permissions. Adds the connect permission as it was missing. Signed-off-by: Hassan Abouelela <[email protected]>
* Fixes Typo in Silence TestsGravatar Hassan Abouelela2020-11-24-1/+1
| | | | Signed-off-by: Hassan Abouelela <[email protected]>
* Move VoiceChat Sync To _unsilenceGravatar Hassan Abouelela2020-11-24-4/+2
| | | | | | Moves the call to voice chat sync from _unsilence_wrapper to _unsilence. Signed-off-by: Hassan Abouelela <[email protected]>
* Moves VoiceChat Sync Out of Overwrites FunctionGravatar Hassan Abouelela2020-11-24-9/+14
| | | | | | | | VoiceChat sync only needs to be called when the command is invoked, instead of while updating permissions. Moved call to command function to reflect that, and fixed failing tests. Signed-off-by: Hassan Abouelela <[email protected]>
* Refractors SendMessage FunctionGravatar Hassan Abouelela2020-11-24-39/+46
| | | | | | | Refractors the send message function in silence to make it more understandable and flexible. Signed-off-by: Hassan Abouelela <[email protected]>
* Fixes Typo in DocGravatar Hassan Abouelela2020-11-24-1/+1
| | | Co-authored-by: Mark <[email protected]>
* Removes AnyChannel ConverterGravatar Hassan Abouelela2020-11-24-91/+3
| | | | | | | Removes the AnyChannel converter in favor of a combination of Text and Voice converters. Signed-off-by: Hassan Abouelela <[email protected]>
* Fixes Voice Channel AccessGravatar Hassan Abouelela2020-11-24-1/+1
| | | | | | | A typo caused the function to return the text channel for `code/help 1`, when it is meant to access `code/help 2`. Signed-off-by: Hassan Abouelela <[email protected]>
* Refractors For Style GuidelinesGravatar Hassan Abouelela2020-11-24-13/+20
| | | | | | | Refractors method signatures and calls to follow python-discord style guide. Signed-off-by: Hassan Abouelela <[email protected]>
* Finalizes Silence & Unsilence UnitTestsGravatar Hassan Abouelela2020-11-23-17/+32
| | | | Signed-off-by: Hassan Abouelela <[email protected]>
* Adds Silence & Unsilence UnitTestsGravatar Hassan Abouelela2020-11-23-1/+211
| | | | Signed-off-by: Hassan Abouelela <[email protected]>
* Removes Redundant Exception HandlingGravatar Hassan Abouelela2020-11-23-38/+20
| | | | Signed-off-by: Hassan Abouelela <[email protected]>
* General Silence Class TestsGravatar Hassan Abouelela2020-11-23-17/+126
| | | | | | Adds tests for helper functions in the silence cog. Signed-off-by: Hassan Abouelela <[email protected]>
* Fixes Voice Silence ReportingGravatar Hassan Abouelela2020-11-23-6/+5
| | | | | | | Fixes the channel reported as muted to voice channel chat channels when silencing voice channels. Signed-off-by: Hassan Abouelela <[email protected]>
* Fix Failing FunctionsGravatar Hassan Abouelela2020-11-23-1/+1
| | | | Signed-off-by: Hassan Abouelela <[email protected]>
* Make Voice Channel Kick OptionalGravatar Hassan Abouelela2020-11-23-16/+34
| | | | | | | | | | Adds an optional parameter to the silence command to enable moderators to choose if they only update permissions, or kick members too. As an accompanying feature, the unsilence command now syncs voice channel permissions too. Signed-off-by: Hassan Abouelela <[email protected]>
* Merge branch 'voicechannel-mute' of https://github.com/HassanAbouelela/bot ↵Gravatar Hassan Abouelela2020-11-23-0/+0
|\ | | | | | | into voicechannel-mute
| * Write AnyChannelConverter TestsGravatar Hassan Abouelela2020-11-23-0/+48
| | | | | | | | Signed-off-by: Hassan Abouelela <[email protected]>
* | Write AnyChannelConverter TestsGravatar Hassan Abouelela2020-11-23-0/+48
|/ | | | Signed-off-by: Hassan Abouelela <[email protected]>
* Add VC Mute Failure NotificationGravatar Hassan Abouelela2020-11-23-21/+24
| | | | | | | Notifies invocation channel that the silence command failed to silence the channel because it could not move members, but roles were updated. Signed-off-by: Hassan Abouelela <[email protected]>
* Add VC Mute FunctionalityGravatar Hassan Abouelela2020-11-22-2/+64
| | | | | | | Adds and calls a function to force a voice channel member to sync permissions. See #1160 for why this is necessary. Signed-off-by: Hassan Abouelela <[email protected]>
* Merge branch 'master' into voicechannel-muteGravatar Hassan Abouelela2020-11-22-430/+589
|\
| * Merge pull request #1204 from python-discord/superstar-fixGravatar Boris Muratov2020-11-21-49/+50
| |\ | | | | | | Modify `!superstar` to use `apply_infraction`.
| | * Empty commitGravatar mbaruh2020-11-21-0/+0
| | |
| | * Merge branch 'master' into superstar-fixGravatar Boris Muratov2020-11-21-1982/+3751
| | |\ | | |/ | |/|
| * | Merge pull request #1287 from python-discord/help-channel-msgGravatar Boris Muratov2020-11-21-10/+25
| |\ \ | | | | | | | | Updated available help channel embed
| | * \ Merge branch 'master' into help-channel-msgGravatar Boris Muratov2020-11-21-371/+514
| | |\ \ | | |/ / | |/| |
| * | | Merge pull request #1293 from ks129/emojis-filterGravatar Daniel Brown2020-11-20-26/+61
| |\ \ \ | | | | | | | | | | Include Unicode emojis to emojis filter
| | * \ \ Merge branch 'master' into emojis-filterGravatar Daniel Brown2020-11-20-345/+453
| | |\ \ \ | | |/ / / | |/| | |
| * | | | Checkout code so we can deployGravatar Sebastiaan Zeeff2020-11-19-0/+9
| | | | |
| * | | | Merge pull request #1291 from python-discord/sebastiaan/backend/fix-deploy-stageGravatar Sebastiaan Zeeff2020-11-19-172/+36
| |\ \ \ \ | | | | | | | | | | | | Fix the deploy stage of our build pipeline
| | * | | | Fix the deploy stage of our build pipelineGravatar Sebastiaan Zeeff2020-11-19-172/+36
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I've fixed the deploy stage of our build pipeline, as it got mixed in with the old workflow file due to a merge conflict. The deploy stage is currently split into a separate workflow; theoretically, this allows us to trigger a redeploy from GitHub, without having to build the container image again. Signed-off-by: Sebastiaan Zeeff <[email protected]>
| * | | | Merge pull request #1286 from python-discord/kubernetes-deployGravatar Joe Banks2020-11-19-3/+195
| |\ \ \ \
| | * \ \ \ Merge branch 'master' into kubernetes-deployGravatar Leon Sandøy2020-11-19-286/+486
| | |\ \ \ \ | | |/ / / / | |/| | | |
| * | | | | Merge pull request #1290 from ↵Gravatar Sebastiaan Zeeff2020-11-19-59/+80
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/sebastiaan/backend/improve-actions-workflow Ensure flake8 actually lints pull requests
| | * | | | | Add comment explaining buildx to workflowGravatar Sebastiaan Zeeff2020-11-19-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's better to document these steps. Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | | | Use GHCR image tags in PipfileGravatar Sebastiaan Zeeff2020-11-18-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | | | Update badges in README to new workflowsGravatar Sebastiaan Zeeff2020-11-18-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | | | Make sure we lint the actual pull requestGravatar Sebastiaan Zeeff2020-11-18-55/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unfortunately, our old setup did not actually lint the PR, as it was running in the context of the target repository. To sidestep the issue of using `pull_request_target` altogether, I've now changed our run of flake8 to using it directly and having it output its errors in teh format of Workflow Commands. This means that our flake8 output will not be translated automatically in annotations for the run. In addition, I've split up the workflow into two separate files: one for linting & testing and one for building (& deploying). Signed-off-by: Sebastiaan Zeeff <[email protected]>
| | * | | | | Pull snekbox image from GHCR in docker-composeGravatar Sebastiaan Zeeff2020-11-18-1/+1
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're in the process of migrating snekbox to the GitHub Container Repository, which will replace DockerHub. I've changed docker-compose to reflect that change. Signed-off-by: Sebastiaan Zeeff <[email protected]>
| * | | | | Merge pull request #1245 - global bot instanceGravatar Mark2020-11-16-282/+304
| |\ \ \ \ \
| | * \ \ \ \ Merge master and fix conflictsGravatar MarkKoz2020-11-16-747/+1885
| | |\ \ \ \ \ | | |/ / / / / | |/| | | | |
| | * | | | | Set up Sentry when running rather than upon importGravatar MarkKoz2020-10-17-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was causing an error if a DSN was not configured. It also feels wrong and confusing to attempt to make a connection just upon import.
| | * | | | | Move bot creation code from __main__.py to bot.pyGravatar MarkKoz2020-10-17-53/+58
| | | | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-30 08:37:47 +0000