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

[Enhancement] Option to not use date filter #30

Closed
jmpoastUP opened this issue Jan 21, 2022 · 3 comments · Fixed by #56
Closed

[Enhancement] Option to not use date filter #30

jmpoastUP opened this issue Jan 21, 2022 · 3 comments · Fixed by #56
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@jmpoastUP
Copy link

Our operations support group would like to, by default, see all open tickets and not select a 'since' date. Can we add an option to not use the date filter? I tried clearing out the date but that didn't work.

@giranm giranm added the enhancement New feature or request label Jan 22, 2022
@jmpoastUP
Copy link
Author

In addition, the date filter doesn't seem to cache like the rest of the filters and we have to re-set it each time we reload the page. Can we get it to cache?

@giranm giranm added this to the Alpha milestone Feb 28, 2022
@giranm
Copy link
Owner

giranm commented Mar 1, 2022

Having reviewed this enhancement, there are a few considerations here:

  • PagerDuty /incidents API endpoint has a default since date of 1 month, going back as far as 6 months.
    No incidents can be returned which are older than 6 months.
  • Querying large number incidents (+1000) in the query can impact user experience; this is a larger issue to be addressed by: [Enhancement] Prompt user if incident query returns a large dataset #52
  • Typically large number of incidents are returned when querying for resolved state; users should consider PagerDuty's core Search for deeper search capabilities on a larger dataset.

Removing the since date filter in the UI will default back to a 1 month lookback, so a good middleground here is to make the since date filter a configurable item.

e.g. Users can set the default lookback to "6M" which means the since data will be set to 6 months before the current date.

@giranm
Copy link
Owner

giranm commented Mar 3, 2022

Given the above feedback and the proposed solution implemented by #56, this issue will now be closed.

@giranm giranm closed this as completed Mar 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants