[GitHub] [superset] jinghua-qa commented on pull request #23512: fix: Dashboard not loading with default first value in filter

["jinghua-qa (via GitHub)" <gi...@apache.org>]

jinghua-qa commented on PR #23512:
URL: https://github.com/apache/superset/pull/23512#issuecomment-1487256638

   > I'm _almost_ sure that that `useEffect` wasn't redundant and something broke when I tried to remove it some time ago. The problem is, I don't remember what it was... Could you try to test some less common scenarios such as opening a dashboard with filters In URL? Maybe @jinghua-qa could help testing this solution? @villebro any chance you remember what that `useEffect` was for?
   
   sure i can test


-- 
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