|  | Commit message (Collapse) | Author | Age | Lines | 
|---|
| | 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| | 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 | 
| | 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| | 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 | 
| | 
| 
| 
| | This also fixes an issue with the `attachments` rule not respecting the most recent message sent by a user. | 
| | |  | 
| | |  | 
| | |  | 
| | |  | 
| |\  
| | 
| | | Add tests for `bot.converters`. | 
| | | |  | 
| |\ \  
| | | 
| | | | Use `@pytest.fixture` for creating contexts. | 
| | | | |  | 
| |\ \ \  
| | | | 
| | | | | Add tests for `bot.cogs.security`. | 
| | | | | |  | 
| |\ \ \ \  
| |/ / /  
|/| | | | Add tests for `bot.cogs.token_remover`. | 
| | | | | |  | 
| |/ / / |  | 
| |\ \ \  
| | | | 
| | | | | Add basic tests for `bot.pagination`. | 
| | |/ / |  | 
| |\ \ \  
| |/ /  
|/| | | Validate bot.cogs.antispam configuration on CI. | 
| | |/ |  | 
| |/ |  | 
| | |  | 
| | |  | 
| | |  | 
|  |  |