redash icon indicating copy to clipboard operation
redash copied to clipboard

Published 20 hours ago • verified publisher icon getredash

Additional Date Ranges for DateRangeParameter

Open rohithsebastian opened this issue 1 year ago • 2 comments

What type of PR is this?

  • [x] Refactor
  • [ ] Feature
  • [ ] Bug Fix
  • [ ] New Query Runner (Data Source)
  • [ ] New Alert Destination
  • [ ] Other

Description

Additional dynamic date ranges for time-based filtering using the DateRangeParameter component.

How is this tested?

  • [ ] Unit tests (pytest, jest)
  • [ ] E2E Tests (Cypress)
  • [x] Manually
  • [ ] N/A

Created new Date Range parameter with new query

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

Screenshot 2024-04-15 at 2 48 26 PM

rohithsebastian avatar Apr 15 '24 18:04 rohithsebastian

Thank you for your contribution. But this solution is not a universal one. I can provide you with an idea to re-develop a relative time component, or add a relative time component based on the existing one. Users can customize this relative time component. For example, Last xx days, months or years.

So I suggest not to merge this branch to master for the time being. 😁

gaecoli avatar Apr 16 '24 02:04 gaecoli

I can provide you with an idea to re-develop a relative time component, or add a relative time component based on the existing one.

That sounds like the better approach. :smile:

justinclift avatar Apr 16 '24 04:04 justinclift