GrafeasGroup / buttercup

Our personal assistant in Discord
MIT License
5 stars 2 forks source link

Replace submission time filters with new version #121

Closed TimJentzsch closed 2 years ago

TimJentzsch commented 2 years ago

Relevant issue: GrafeasGroup/blossom#228

Description:

This replaces the 'from' query parameter with 'complete_time__gte' as well as 'until' with 'complete_time__lte'.

Directly depends on GrafeasGroup/blossom#228, which needs to be merged and deployed first. While the new system is deployed and this PR not merged, most Buttercup commands won't work with time filters.

Note: This needs to be tested with the new deployed time filters before merging!

Testing Instructions:

All commands need to be tested with time filters. It needs to be ensured that the commands don't throw any new errors and that the correct time frames are filtered. This should be tested with both absolute and relative commands.

Checklist:

TimJentzsch commented 2 years ago

Ok, this seems to work well