aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | * | Fix AttributeError with cog special methodsGravatar MarkKoz2019-09-25-4/+4
| |/ / | | | | | | | | | | | | | | | | | | Discord.py's internals use the __func__ attribute of special methods (cog_command_error, cog_check, cog_before_invoke, cog_after_invoke). Therefore the methods must be bound methods rather than static so that the attribute exists.
| * / Note Type CorrectionGravatar Daniel Brown2019-09-24-1/+1
|/ / | | | | | | | | | | - In the database, notes were being listed as "warnings" despite having a type specifically for them. Changed it so that notes are now listed as the proper type. Signed-off-by: Daniel Brown <[email protected]>
* | Merge pull request #448 from Akarys42/ot-fixGravatar S. Co12019-09-24-3/+8
|\ \ | | | | | | Catch error in case of non successfull API call for new ot-names
| * \ Merge branch 'master' into ot-fixGravatar S. Co12019-09-24-171/+166
| |\ \ | |/ / |/| |
* | | Merge pull request #451 from python-discord/all-the-shieldsGravatar Mark2019-09-24-3/+7
|\ \ \ | | | | | | | | Add all the shields!
| * | | Add all the shieldsGravatar scragly2019-09-25-3/+7
|/ / /
* | | Merge pull request #450 from python-discord/rich-embed-false-positive-fixGravatar S. Co12019-09-24-15/+33
|\ \ \ | | | | | | | | Prevent false-positives of the rich embed filter
| * \ \ Merge branch 'master' into rich-embed-false-positive-fixGravatar S. Co12019-09-24-153/+126
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #443 from Akarys42/masterGravatar Mark2019-09-24-11/+8
|\ \ \ \ | | | | | | | | | | Change log.error to log.exception
| * \ \ \ Merge branch 'master' into masterGravatar Mark2019-09-24-231/+286
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Docker Build & CI Refinements (#444)Gravatar scragly2019-09-25-142/+118
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Docker Build & CI Refinements Co-authored-by: S. Co1 <[email protected]>
| * | | | | Remove unnecessary sudo for pipenv install.Gravatar scragly2019-09-25-1/+1
| | | | | |
| * | | | | Fix reminders filter queryGravatar S. Co12019-09-24-2/+2
| | | | | | | | | | | | | | | | | | Closes #445
| * | | | | Remove regex wheel as dateparser is no longer used.Gravatar scragly2019-09-25-0/+0
| | | | | |
| * | | | | Create default django admin user.Gravatar scragly2019-09-24-1/+3
| | | | | |
| * | | | | Fix random text mistake in Dockerfile.Gravatar scragly2019-09-24-1/+1
| | | | | |
| * | | | | Add docker build condition for CI.Gravatar scragly2019-09-24-0/+1
| | | | | |
| * | | | | Remove unneeded Pipfile scripts and azure stages/vars.Gravatar scragly2019-09-24-51/+14
| | | | | |
| * | | | | Add a dev-only compose for new devs who don't have `site`.Gravatar scragly2019-09-24-0/+28
| | | | | |
| * | | | | Pin dependencies, pre-build `regex` pkg.Gravatar scragly2019-09-24-20/+17
| | | | | |
| * | | | | Reformat indents, use Docker@2 task for azure instead of sh script.Gravatar scragly2019-09-24-78/+63
|/ / / / /
| * | | | Fix E128 linting errorGravatar Akarys422019-09-23-4/+4
| | | | |
| * | | | Merge branch 'python-discord-master'Gravatar Akarys422019-09-23-4931/+5537
| |\ \ \ \
| | * | | | Merge branch 'master' of https://github.com/python-discord/bot into ↵Gravatar Akarys422019-09-23-4931/+5537
| |/| | | | | | | | | | | | | | | | | | | | | | python-discord-master
| * | | | | Change log.error to log.exceptionGravatar Akarys422019-09-22-10/+7
| | | | | | | | | | | | | | | | | | | | | | | | See issue #382
| | | * | | Prevent false-positives of the rich embed filterGravatar Sebastiaan Zeeff2019-09-24-15/+33
| |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/python-discord/bot/issues/293 The rich embed filter is plagued by false positives now Discord has added more custom preview embeds for various websites. Since these embeds have the `rich` type instead of the `link` type, these embeds triggered the filter we had in place. This commit remedies that by using the existing URL regex pattern to list all the URLs contained in the message content and then checking if the embed url is a member of that list. If so, it's very likely that the embed was auto-generated from that URL, so we should ignore it. This approach deviates slightly from that outlined in #293. This does increase the probability of a false-negative, as a "true" user-generated rich embed could also have a url that's contained in the message body. However, I've checked most of the triggers we have had in the past and none of the legitimate triggers would have been a false-negative under the new rules. Therefore, I think it's very reasonable to adopt this strategy. In addition to the change in behavior of the rich embed filter, I have also kaizened the existing regex patterns by compiling them at load time. Since we check a lot of regex patterns for every message received by the bot, this should be beneficial for performance.
| | | * | Remove repeat logic for off-topic-name api callGravatar Sebastiaan Zeeff2019-09-24-6/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After a short discussion in the core-dev team, we decided to not use retry logic for a failed API call for new off-topic-names. We may introduce something similar in the future, but we're not sure on the direction we want to take yet.
| | | * | Put import in alphabetical orderGravatar Matteo Bertucci2019-09-24-1/+1
| | | | |
| | | * | Override waiting time to half an hour Gravatar Matteo Bertucci2019-09-24-1/+6
| | | | | | | | | | | | | | | If an exception occurred
| | | * | Fix importing errorGravatar Matteo Bertucci2019-09-24-1/+2
| | | | | | | | | | | | | | | `bot.api.ResponseCodeError` is now imported
| | | * | Merge branch 'master' into ot-fixGravatar Matteo Bertucci2019-09-24-90/+169
| | | |\ \ | |_|_|/ / |/| | | |
* | | | | Merge pull request #449 from python-discord/duration-converter-fixGravatar Sebastiaan Zeeff2019-09-24-90/+169
|\ \ \ \ \ | |_|/ / / |/| | | | Fix failing duration conversion
| * | | | Restructure Duration converter docstringGravatar Sebastiaan Zeeff2019-09-23-4/+10
| | | | | | | | | | | | | | | | | | | | | | | | | In this commit, I've restructured the Duration converter's docstring to a more readable format: a bullet point list.
| * | | | Allow whitespace in duration strings and update testsGravatar Sebastiaan Zeeff2019-09-23-123/+87
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/python-discord/bot/issues/446 After review feedback and a discussion in the dev-core team, I've changed a couple of things: - Allow a space between amount and unit in the duration string; - Allow a space between different units in the duration string; - Remove the old ExpirationDate converter completely; - Remove the dependency `dateparser` from the Pipfile; - Update tests for the two types of optional spaces; - Change the test for valid cases to a more readable format; This PR closes #446
| * | | | Fix failing duration conversionGravatar Sebastiaan Zeeff2019-09-23-18/+127
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/python-discord/bot/issues/446 The current ExpirationDate converter does not convert duration strings to `datetime.datetime` objects correctly. To remedy the problem, I've written a new Duration converter that uses regex matching to extract the relevant duration units and `dateutil.relativedelta.relativedelta` to compute a `datetime.datetime` that's the given duration in the future. I've left the old `ExpirationDate` converter in place for now, since the new Duration converter may not be the most optimal method. However, given the importance of being able to convert durations for moderation tasks, I think it's better to implement Duration now and rethink the approach later. This commit closes #446
| | * | Add `continue` in the except blockGravatar Matteo Bertucci2019-09-24-0/+1
| | | |
| | * | Catch error in case of non successfull API call for new ot-namesGravatar Akarys422019-09-23-3/+6
| |/ / |/| | | | | | | | This allows to keep the task running even if the call fail.
* | | Make DEFCON days subcommand enable DEFCON (#405)Gravatar scragly2019-09-23-66/+70
|\ \ \ | | | | | | | | | | | | | | | | | | | | Make DEFCON days subcommand enable DEFCON Co-authored-by: Joseph <[email protected]> Co-authored-by: null <[email protected]>
| * \ \ Merge branch 'master' into defcon-fix-djangoGravatar scragly2019-09-23-1708/+1066
| |\ \ \ | |/ / / |/| | |
* | | | Update linting (#406)Gravatar scragly2019-09-23-1708/+1066
|\ \ \ \ | |_|_|/ |/| | | Update linting
| * | | Apply suggestions from code reviewGravatar sco12019-09-22-235/+156
| | | | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]>
| * | | Apply suggestions from code reviewGravatar S. Co12019-09-22-29/+39
| | | | | | | | | | | | Co-Authored-By: Mark <[email protected]>
| * | | Merge branch 'master' into flake8-pluginsGravatar sco12019-09-22-3/+3
| |\ \ \ | |/ / / |/| | |
| * | | Apply suggestions from code reviewGravatar S. Co12019-09-22-3/+3
| | | | | | | | | | | | Co-Authored-By: Leon Sandøy <[email protected]>
| * | | Apply suggestions from code reviewGravatar sco12019-09-19-2/+2
| | | | | | | | | | | | | | | | Co-Authored-By: Sebastiaan Zeeff <[email protected]>
| * | | Apply suggestions from code reviewGravatar S. Co12019-09-19-4/+4
| | | | | | | | | | | | Co-Authored-By: Sebastiaan Zeeff <[email protected]>
| * | | Update imports for applied code review suggestionsGravatar sco12019-09-19-4/+3
| | | |
| * | | Apply suggestions from code reviewGravatar S. Co12019-09-19-13/+13
| | | | | | | | | | | | Co-Authored-By: Sebastiaan Zeeff <[email protected]>
| * | | Replace incorrect None returns with Optional in certain commandsGravatar sco12019-09-19-17/+24
| | | |
| * | | Use Optional typing shorthand rather than Union w/NoneGravatar sco12019-09-19-6/+6
| | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-26 16:36:22 +0000