| Commit message (Collapse) | Author | Lines |
|
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]>
|
|
Restructures some code to make it more understandable and reduce
duplication.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Refactors the send message utility function to make it more legible and
reduce unnecessary calls.
Co-authored-by: Mark <[email protected]>
|
|
Reduces redundant calls to isinstance by saving the result where
applicable.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Replaces a repeated hardcoded message with a dynamically built one.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
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]>
|
|
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Moves the call to voice chat sync from _unsilence_wrapper to _unsilence.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
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 the send message function in silence to make it more
understandable and flexible.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Co-authored-by: Mark <[email protected]>
|
|
Removes the AnyChannel converter in favor of a combination of Text and
Voice converters.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
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 method signatures and calls to follow python-discord style
guide.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Adds tests for helper functions in the silence cog.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Fixes the channel reported as muted to voice channel chat channels when
silencing voice channels.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
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]>
|
|
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
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]>
|
|
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]>
|
|
Refactors imports of silence class to be more inline with the original
import structure.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
- Allows spaces in channel name
- Allows channel name to have any capitalization
- Fixed inherited class to general Converter class
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
- This removes a duplicate checkmark from the title of the embed
- The checkmark was left from the previous title system and wasn't removed in the change
|
|
- This changes "we'll try to help you" to say "others will try to help you"
- Clarifies that the rest of the community is going to help — not some dedicated help/staff team
|
|
|
|
Instead detecting only custom emojis, rule now catch too Unicode emojis.
This converts Unicode emojis to :emoji: format and count them.
|
|
|
|
|
|
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]>
|
|
|
|
It's better to document these steps.
Signed-off-by: Sebastiaan Zeeff <[email protected]>
|
|
Signed-off-by: Sebastiaan Zeeff <[email protected]>
|
|
Signed-off-by: Sebastiaan Zeeff <[email protected]>
|
|
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]>
|
|
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]>
|
|
- This replaces "one" with "any"
- This is supposed to read better
|
|
- Allows the icon to be centered
|
|
|
|
- Adds a footer and title
- Uses a green colored embed
- Updates message to be easier to read and contain practical info for asking better questions
|
|
- The color is used in the new help channel embed
|
|
Adds an optional channel parameter to silence and unsilence commands,
and adds ability to silence voice channels.
TODO: New Tests
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
|
|
|