fix(optimiser): Prevent invalid time range#4113
fix(optimiser): Prevent invalid time range#4113sevensxyt wants to merge 2 commits intonusmodifications:masterfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@sevensxyt is attempting to deploy a commit to the modsbot's projects Team on Vercel. A member of the Team first needs to authorize it. |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4113 +/- ##
==========================================
- Coverage 54.52% 52.91% -1.62%
==========================================
Files 274 289 +15
Lines 6076 6696 +620
Branches 1455 1633 +178
==========================================
+ Hits 3313 3543 +230
- Misses 2763 3153 +390 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Hi @sevensxyt, thanks for the PR. I'm currently working on a refactor of the optimiser component in #4094, and it also modifies the time select. I'll update you again when those changes are ready. |
|
Hi @sevensxyt, #4094 has been merged. Let me know if you are still interested in working on this. Thanks for your patience and sorry for the delay! |
Context
On the optimiser page, time options for the dropdown lists are not filtered. This allows invalid time ranges to be selected, where the start time is after the end time, as shown below:
Start time
18:00is after end time14:00Implementation
Other Information
I've also added tests for the utility functions added