-
Notifications
You must be signed in to change notification settings - Fork 21
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
Configurable date range filter for entity lists #1487
Configurable date range filter for entity lists #1487
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.
Let's try to make more use of moments flexibility so we don't have to predefine all the units ourselves. Besides that looks good. Especially with the tests!
src/app/core/entity-components/entity-list/filter-generator.service.ts
Outdated
Show resolved
Hide resolved
…ange-filter-for-Entity-Lists
…ange-filter-for-Entity-Lists
Co-authored-by: Sebastian <sleidig@users.noreply.github.com>
…ange-filter-for-Entity-Lists
…Lists # Conflicts: # package-lock.json
…ange-filter-for-Entity-Lists # Conflicts: # src/app/core/entity-components/entity-list/EntityListConfig.ts
…Lists # Conflicts: # src/app/app.module.ts # src/app/core/entity-components/entity-details/entity-details.module.ts # src/app/core/entity-components/entity-list/EntityListConfig.ts # src/app/core/filter/filter.module.ts
…ange-filter-for-Entity-Lists
…ange-filter-for-Entity-Lists
…ange-filter-for-Entity-Lists
Deployed to https://pr-1487.aam-digital.net/ |
…date range filter panel
… to test the constructor
The test for the indexOfCorrespondingDateRange is quite incomplete and inadequate, but apart from that this PR could probably soon be reviewed, @TheSlimvReal. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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.
Great work. The UI and functionality is very clean. I am looking forward to releasing this!
🎉 This PR is included in version 3.22.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Date range filters can now be configured via the
config-fix.ts
, as described in #1448.Configuration via UI, as described in #1383 is also implemented.