| Commit message (Collapse) | Author | Age | Lines | |
|---|---|---|---|---|
| * | Allow whitespace in duration strings and update tests |  Sebastiaan Zeeff | 2019-09-23 | -47/+66 |
| | | | | | | | | | | | | | | | | | | | | | | 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 conversion | Sebastiaan Zeeff | 2019-09-23 | -4/+78 |
| | | | | | | | | | | | | | | | | | | | | 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 tests for `bot.converters`. |  Johannes Christ | 2019-09-15 | -0/+93 |
 |
index : bot | |
| A mirrored copy of the python-discord/bot repository. |
| aboutsummaryrefslogtreecommitdiffstats |