kokhlo commented on issue #43385:
URL: https://github.com/apache/superset/issues/43385#issuecomment-5462595545
Taking this one. Quick triage against master: the list-payload perf PRs
referenced above (#29121, #28609, #38567, #36246) are all already merged, and
`PUT /api/v1/dashboard/<pk>` is O(1) per dashboard on master — so the part
that's reproducibly wrong on master today is the UX gap: `+ Dashboard` fires a
hard `window.location.assign('/dashboard/new/')` (DashboardList/index.tsx) with
no pending state, and that GET creates the dashboard server-side before
redirecting to the editor — so the click looks dead for the entire round-trip.
I'll push a fix that puts the create button (both on the Dashboards list and
the home table) into a loading state before the navigation, with unit tests.
--
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]
