imddevaraj commented on PR #41527:
URL: https://github.com/apache/superset/pull/41527#issuecomment-4842614049

   > @yousoph @kasiazjc thoughts on this feature?
   > 
   > @imddevaraj wondering if we can make this more generic? Like right now we 
are adding some defaults but the list gets bigger as others may want more like, 
what if i wanted last 4 hours, 24 hours, 8 hours, etc?
   
   @sadpandajoe 
   Good suggestion. I've updated the implementation to make it more generic. 
Instead of supporting only the initial default ranges, the quick-range list is 
now configurable, making it easy to add additional options such as Last 4 
Hours, Last 8 Hours, Last 24 Hours, or any other commonly requested time ranges 
in the future without further code changes.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to