ktmud opened a new issue #12508: URL: https://github.com/apache/superset/issues/12508
### Expected results In 0.38 and earlier, if you set "30 days" or "5 months" in start date, it will be automatically converted to the equivalent of "30 days ago". The new time picker will treat it as "30 days after today", causing an `Unexpected time range: From date cannot be larger than to date` error for some existing charts. Time ranges that break in the new time picker but valid for the old one: 1. "30 days: " -> "30 days ago : now" 3. "7 days: 7 days" -> "7 days ago : 7 days after today" ### Actual results Existing charts with "x periods" set in start date should behave the same. Since the original behavior is a little bit ambiguous, we may actually want to force users to use "30 days ago" or "- 30 days" for this use case. So I think it's OK to keep current logic, but to minimize disruption, we should do a db migration to automatically fix such charts and add a breaking change warning in the `v1.0` changelog (albeit numbers users who used this syntax is low). ### Environment Latest master ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
