[GitHub] [superset] villebro edited a comment on pull request #17039: [WIP] feat(config): Add none force-refresh option for auto refresh

[GitBox <gi...@apache.org>]

villebro edited a comment on pull request #17039:
URL: https://github.com/apache/superset/pull/17039#issuecomment-939879154


   I agree with @geido that this should be a general config, not a feature flag. I would propose adding something similar to this to `config.py`:
   
   ```python
   DASHBOARD_AUTO_REFRESH_MODE: Literal["fetch", "force"] = "force"
   ```
   
   Then that needs to be added to bootstrap data by adding the key to `FRONTEND_CONF_KEYS` on the backend. After this the value should be available on the dashboard on `dashboardInfo.common.conf`. But I may be oversimplifying things - let me know if you need help @yongchand!


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org