Raphael Krut-Landau comments

Results 43 comments of


                                            Raphael Krut-Landau

Avoid race condition in Browse models filter

> [!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....

Avoid race condition in Browse models filter

Closing for now. Will return to this if the priority is determined to be higher.

[wip] improve loading wrapper

> [!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....

[wip] Question config form with better loading

> [!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....

Tweaks to appearance of cache page and form

* **#42856** 👈 * `master` This stack of pull requests is managed by Graphite. Learn more about stacking. Join @rafpaf and the rest of your teammates on Graphite

Questions always say "Updated a few seconds ago"

Heads up that we may just be getting rid of this message https://metaboat.slack.com/archives/C06KX7QECN4/p1715972915986779?thread_ts=1715850000.658139&channel=C06KX7QECN4&message_ts=1715972915.986779

fix(admin/performance): Ensure that adding or removing default values doesn't make cache policy form count as dirty

* **#46543** 👈 * `master` This stack of pull requests is managed by Graphite. Learn more about stacking. Join @rafpaf and the rest of your teammates on Graphite

fix(admin/performance): Ensure that adding or removing default values doesn't make cache policy form count as dirty

### Merge activity * **Aug 28, 10:10 AM EDT**: @rafpaf merged this pull request with [Graphite](https://app.graphite.dev/github/pr/metabase/metabase/46543).

Remove withinPortal console.error

* **#51938** 👈 (View in Graphite) * `master` This stack of pull requests is managed by Graphite. Learn more about stacking.

Add simple confirmation modal to dictionary upload

* **#59479** 👈 (View in Graphite) * **#56879** : 20 other dependent PRs ([#56557](https://github.com/metabase/metabase/pull/56557) , [#56880](https://github.com/metabase/metabase/pull/56880) , [#57481](https://github.com/metabase/metabase/pull/57481) and 17 others) * **#57534** * `master` This stack of pull requests...