-
-
Notifications
You must be signed in to change notification settings - Fork 825
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
Add freeform relative date for 'This Fiscal Year' #14894
Conversation
(Standard links)
|
f586026
to
1b12e13
Compare
1b12e13
to
e34c927
Compare
@MegaphoneJon is this one you can look at? |
Just returned from a holiday. Will look into this later today. |
@MegaphoneJon @magnolia61 can I get you to check this out? |
I was finally able to look at this. The mechanics work fine and as expected.
The labels in the options would correspond like this
Indeed I used the names differently before (this_1.fiscal_year for this and 1 previous). But I think what you propose is more logical. |
Merging based on @magnolia61 review - thanks of that! |
this_6.fiscal_year - This and previous 5 fiscal years (note 5 here, not 6) Or 6 most recent fiscal years (including the current one) |
Overview
Added support for this_n.year & this_n.fiscal_year
Before
No support for 'Last year & this year' or 'Last fiscal year & this fiscal year' ('These 2 years')
After
Support at the code level - as with #12682 it is necessary to add an option value to the relative_date_filters custom group to expose any new options
Technical Details
@magnolia61 I decided to add a test for yours to finish it - but I found I had a slightly different assumption ie
this_2.fiscal_year = 'these 2 fiscal years' - ie. last year & this year. I think your assumption was that this_2 would actually be 2 years.
Comments