Do not use history.goBack() to trigger navigation (#34956)
- Some tests had a redundant initial route just so that there is an entry in history we can go back to. This caused NewItemMenu to be redundantly rendered, which caused race conditions in the entity framework. We do not need history.goBack(), we can use history.push() but we need an extra, safe route we can try to navigate to. - @see https://metaboat.slack.com/archives/C057T1QTB3L/p1697183111294489 - @see https://metaboat.slack.com/archives/C5XHN8GLW/p1697811520881169 - @see https://github.com/metabase/metabase/pull/34661
Loading
Please register or sign in to comment