metabase icon indicating copy to clipboard operation
metabase copied to clipboard

[wip] improve loading wrapper

Open rafpaf opened this issue 1 year ago • 1 comments

[!IMPORTANT] For those employed by Metabase: if you are merging into master, please add either a backport or a no-backport label to this PR. You will not be able to merge until you do this step. Refer to the section Do I need to backport this PR? in the Metabase Branching Strategy document for more details. If you're not employed by Metabase, this section does not apply to you, and the label will be taken care of by your reviewer.

Warning

If that is your first contribution to Metabase, please sign the Contributor License Agreement (unless it's a tiny documentation change). Also, if you're attempting to fix a translation issue, please submit your changes to our POEditor project instead of opening a PR.

Closes https://github.com/metabase/metabase/issues/[issue_number]

Description

Describe the overall approach and the problem being solved.

How to verify

Describe the steps to verify that the changes are working as expected.

  1. New question -> Sample Dataset -> ...
  2. ...

Demo

Upload a demo video or before/after screenshots if sensible or remove the section

Checklist

  • [ ] Tests have been added/updated to cover changes in this PR

rafpaf avatar May 16 '24 21:05 rafpaf

[!WARNING] This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite. Learn more

  • #42806 Graphite
  • #42805 Graphite 👈
  • #42729 Graphite: 1 other dependent PR (#42788 Graphite)
  • master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rafpaf and the rest of your teammates on Graphite Graphite

rafpaf avatar May 16 '24 21:05 rafpaf