Skip to content
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

Fix "Older than" relative date inputs #102

Merged
merged 2 commits into from
Apr 8, 2021
Merged

Fix "Older than" relative date inputs #102

merged 2 commits into from
Apr 8, 2021

Conversation

schlick
Copy link
Member

@schlick schlick commented Apr 1, 2021

The "older than" relative date options are often used to exclude records that occur on a weekly or fortnightly basis. For example, for events that occur every Monday you could use the "Older than 7 days" option to generate alerts for ones that are overdue, not due today (via a last occurred on date). If the current date is Monday 29/03/21 then 7 days ago is Monday 22/03/21. You don't want to include the events that took place on that previous Monday. So you need the filter to be more than 7 days ie Sunday 21/03/21. So "Older than 7 days" ought to use an exclusive to date.

@schlick schlick changed the title [NEP-10976] Fix "Older than" relative date inputs Fix "Older than" relative date inputs Apr 6, 2021
@schlick schlick merged commit ef2b258 into develop Apr 8, 2021
@schlick schlick deleted the NEP-10976 branch April 8, 2021 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants