-
Notifications
You must be signed in to change notification settings - Fork 58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes to schedule all reports, even for once #3840
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work, looks good to me 👍 The PR is quite detailed, thanks for that. Just a small note for the next time: please remove the boilerplate text (in the "Issue link" section, either fill it in or remove the section entirely)
There's still an issue with the tests. Apparently a |
Yes, this will be fixed probably when the Pr of JP is merged with main, also the test fix :) |
Co-authored-by: Peter-Paul van Gemerden <[email protected]>
Co-authored-by: Peter-Paul van Gemerden <[email protected]>
We continue with implementation of time widget as per: |
Changes
deadline_at
(start date and time)auto_calculate_deadline
attribute to Scheduler #3869Issue link
#3753
Closes #3753
Demo
QA notes
You will need to restart the report runner.
Code Checklist
.env
changes files if required and changed the.env-dist
accordingly.Checklist for code reviewers:
Copy-paste the checklist from the docs/source/templates folder into your comment.
Checklist for QA:
Copy-paste the checklist from the docs/source/templates folder into your comment.