-
- Downloads
Optimize DB calls of dashboard card updates some more (#29132)
* Optimize average case DB calls * Typo * Fix test * Remove duplicate require * Return nil * Fixes * Fix docstring * Remove unused require * Avoid hydrating ordered_cards on the dashboard again * Tidy * Update docstring * Move transaction * Move comment * Remove dashboard->param-field-ids * whitespace * Tidy update-dashcards! * Updates from suggestions * Update test * Update test description Co-authored-by:Ngoc Khuat <qn.khuat@gmail.com> --------- Co-authored-by:
Ngoc Khuat <qn.khuat@gmail.com>
Showing
- src/metabase/api/dashboard.clj 5 additions, 5 deletionssrc/metabase/api/dashboard.clj
- src/metabase/api/public.clj 4 additions, 2 deletionssrc/metabase/api/public.clj
- src/metabase/models/dashboard.clj 28 additions, 19 deletionssrc/metabase/models/dashboard.clj
- src/metabase/models/dashboard_card.clj 27 additions, 26 deletionssrc/metabase/models/dashboard_card.clj
- src/metabase/models/params.clj 22 additions, 22 deletionssrc/metabase/models/params.clj
- test/metabase/models/dashboard_card_test.clj 56 additions, 15 deletionstest/metabase/models/dashboard_card_test.clj
Loading
Please register or sign in to comment