aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | Source: Remove `command` shadowing on converterGravatar ks1292020-05-20-3/+3
| | | | | | | | |
| * | | | | | | | Source: Make `source` command to `command` instead `group`Gravatar ks1292020-05-20-2/+2
| | | | | | | | |
| * | | | | | | | Source: Create `source` command with alias `src`Gravatar ks1292020-05-20-1/+7
| | | | | | | | |
| * | | | | | | | Source: Create `build_embed` function that build embed of source itemGravatar ks1292020-05-20-0/+27
| | | | | | | | |
| * | | | | | | | Source: Add cog loading to __main__.pyGravatar ks1292020-05-20-0/+1
| | | | | | | | |
| * | | | | | | | Source: Create `get_source_link` function that build item's GitHub linkGravatar ks1292020-05-20-2/+23
| | | | | | | | |
| * | | | | | | | Source: Create converter for source object convertingGravatar ks1292020-05-19-2/+31
| | | | | | | | |
| * | | | | | | | Source: Created initial cog layout + setup functionGravatar ks1292020-05-19-0/+15
| | | | | | | | |
* | | | | | | | | Merge pull request #1055 from python-discord/Senjan/clean_messageGravatar Mark2020-07-20-1/+34
|\ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / |/| | | | | | | | Added command&system to purge all messages up to given message
| * | | | | | | | Merge branch 'master' into Senjan/clean_messageGravatar Senjan212020-07-16-1/+8
| |\ \ \ \ \ \ \ \
| * | | | | | | | | Prevent deleting messages above the desired message.Gravatar Senjan212020-07-16-6/+7
| | | | | | | | | |
| * | | | | | | | | Fix docstring and commentsGravatar Senjan212020-07-15-13/+7
| | | | | | | | | |
| * | | | | | | | | Added command&system to purge all messages up to given messageGravatar Senjan212020-07-15-1/+39
| | | | | | | | | |
| | | | | * | | | | Edited tests to reflect changes (removed py formatting)Gravatar wookie1842020-07-20-3/+3
| | | | | | | | | |
| | | | | * | | | | Removed python formatting from returned codeblockGravatar wookie1842020-07-20-1/+1
| |_|_|_|/ / / / / |/| | | | | | | |
* | | | | | | | | Merge pull request #993 from python-discord/kwzrd/incidentsGravatar kwzrd2020-07-17-12/+1203
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ |/| | | | | | | | Implement #incidents channel automation
| * | | | | | | | Merge branch 'master' into kwzrd/incidentsGravatar kwzrd2020-07-17-1/+1
| |\ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
* | | | | | | | | Bugfix: Show ID for embed DM relays, too.Gravatar Leon Sandøy2020-07-16-1/+1
| | | | | | | | |
| * | | | | | | | Incidents: reduce log level of 403 exceptionGravatar kwzrd2020-07-16-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In addition to 404, this shouldn't send Sentry notifs. Co-authored-by: MarkKoz <[email protected]>
| * | | | | | | | Merge 'master' into 'kwzrd/incidents'; resolve config conflictsGravatar kwzrd2020-07-16-351/+626
| |\ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add upstream changes & resolve config conflicts introduced by #1041. In the conflicting parts of the diff, this commit also re-sorts constants which were added upstream to lexicographical order. This does **not** re-sort all constants added upstream.
* | | | | | | | | Merge pull request #1046 from ItsCinnabar/patch-1Gravatar Sebastiaan Zeeff2020-07-16-1/+1
|\ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / |/| | | | | | | | Update or-gotcha.md
| * | | | | | | | Merge branch 'master' into patch-1Gravatar Sebastiaan Zeeff2020-07-16-1/+8
| |\ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
* | | | | | | | | Merge pull request #1045 from python-discord/bug/980/fuzzy-processingGravatar Dennis Pham2020-07-15-1/+8
|\ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / |/| | | | | | | | Invoke fuzzywuzzy's processor before matching
| * | | | | | | | Merge branch 'master' into bug/980/fuzzy-processingGravatar Dennis Pham2020-07-15-177/+427
| |\ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
| * | | | | | | | Pass processed string to `extractBests`Gravatar kosayoda2020-07-15-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes a regression where the string to be matched was not processed beforehand.
| * | | | | | | | Invoke fuzzywuzzy's processor before matchingGravatar kosayoda2020-07-14-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Trying to match a string with only non-alphanumeric characters results in a warning by fuzzywuzzy. Processing the string before matching lets us avoid the warning, which which uses the root logger and thus isn't supressible.
| | * | | | | | | Merge branch 'master' into patch-1Gravatar Dennis Pham2020-07-15-175/+425
| | |\ \ \ \ \ \ \ | |_|/ / / / / / / |/| | | | | | | |
* | | | | | | | | Merge pull request #1054 from python-discord/dm_relayGravatar Leon Sandøy2020-07-15-6/+43
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | DM relay - minor fixes
| * \ \ \ \ \ \ \ \ Merge branch 'master' into dm_relayGravatar Leon Sandøy2020-07-15-70/+202
| |\ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | |
| * | | | | | | | | Oops, these prints shouldn't be here.Gravatar Leon Sandøy2020-07-15-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/python-discord/bot/issues/1041
| * | | | | | | | | How did that except except block get in?Gravatar Leon Sandøy2020-07-15-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Weird. https://github.com/python-discord/bot/issues/1041
| * | | | | | | | | Create a UserMentionOrID converter.Gravatar Leon Sandøy2020-07-15-1/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we're using the !reply command, using a regular UserConverter is somewhat problematic. For example, if I wanted to send the message "lemon loves you", then I'd try to write `!reply lemon loves you` - however, the optional User converter would then try to convert `lemon` into a User, which it would successfully do since there's like 60 lemons on our server. As a result, the message "loves you" would be sent to a user called lemon.. god knows which one. To solve this bit of ambiguity, I introduce a new converter which only converts user mentions or user IDs into User, not strings that may be intended as part of the message you are sending. https://github.com/python-discord/bot/issues/1041
| * | | | | | | | | Ves' refactorGravatar Leon Sandøy2020-07-15-15/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Sebastiaan Zeeff <[email protected]>
| * | | | | | | | | Add some stats for DMs sent and received.Gravatar Leon Sandøy2020-07-15-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/python-discord/bot/issues/1041
| * | | | | | | | | Add the user ID to the username in dm relays.Gravatar Leon Sandøy2020-07-15-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without this, it is difficult to know precisely who the user that is DMing us is, which might be useful to us. https://github.com/python-discord/bot/issues/1041
| * | | | | | | | | Restore DM user caching.Gravatar Leon Sandøy2020-07-15-6/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 042f472a
| | | * | | | | | | Update bot/resources/tags/or-gotcha.mdGravatar ItsCinnabar2020-07-14-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Sebastiaan Zeeff <[email protected]>
| | | * | | | | | | Update or-gotcha.mdGravatar ItsCinnabar2020-07-14-1/+0
| | | | | | | | | |
| | | * | | | | | | Update or-gotcha.mdGravatar ItsCinnabar2020-07-14-1/+1
| | | | | | | | | |
| | | * | | | | | | Update or-gotcha.mdGravatar ItsCinnabar2020-07-14-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adjust description and include link to docs
| | | | * | | | | | Incidents: reduce log level of 404 exceptionGravatar kwzrd2020-07-16-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: MarkKoz <[email protected]>
| | | | * | | | | | Incidents: link `proxy_url` if attachment fails to downloadGravatar kwzrd2020-07-14-50/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Suggested by Mark during review. If the download fails, we fallback on showing an informative message, which will link the attachment cdn link. The attachment-handling logic was moved from the `archive` coroutine into `make_embed`, which now also returns the file, if available. In the end, this appears to be the smoothest approach. Co-authored-by: MarkKoz <[email protected]>
| | | | * | | | | | Incidents: add `download_file` helper & testsGravatar kwzrd2020-07-13-0/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: MarkKoz <[email protected]>
| | | | * | | | | | Merge d.py version bump from master branchGravatar kwzrd2020-07-10-3/+3
| | | | |\ \ \ \ \ \
| | | | * | | | | | | Config: refactor #incidents constants to lexicographical sortingGravatar kwzrd2020-07-09-12/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: MarkKoz <[email protected]>
| | | | * | | | | | | Incidents tests: improve in-line comment wordingGravatar kwzrd2020-07-09-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: MarkKoz <[email protected]>
| | | | * | | | | | | Incidents: use `moderation_roles` constantGravatar kwzrd2020-07-09-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Better than building the set manually. Tested against regression by comparing the two sets for equality. Suggested by vivax. Co-authored-by: vivax3794 <[email protected]>
| | | | * | | | | | | Merge 'master' into 'kwzrd/incidents'; resolve mod pkg init conflictGravatar kwzrd2020-07-08-217/+672
| | | | |\ \ \ \ \ \ \
| | | | * | | | | | | | Incidents: archive incident attachmentsGravatar kwzrd2020-07-03-1/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is no handling of file types as explained in the `archive` docstring. Testing indicates that relaying incidents with e.g. a text file attachment is simply a noop in the Discord GUI. If there is at least one attachment, we always only relay the one at index 0, as it is believed the user-sent messages can only contain one attachment at maximum. This also adds an extra test asserting the behaviour when an incident with an attachment is archived. The existing test for `archive` is adjusted to assume no attachments. Joe helped me conceive & test this. Co-authored-by: Joseph Banks <[email protected]>
| | | | * | | | | | | | Incidents: trace-level log incident embed creationGravatar kwzrd2020-07-02-0/+2
| | | | | | | | | | | |