-
Notifications
You must be signed in to change notification settings - Fork 5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Question + Model Picker #40207
Question + Model Picker #40207
Conversation
|
495bccf
to
e6c5112
Compare
e6c5112
to
c1d45e8
Compare
c1d45e8
to
7634725
Compare
frontend/src/metabase/models/containers/ModelDetailPage/ModelDetailPage.tsx
Outdated
Show resolved
Hide resolved
d565856
to
c45a1af
Compare
Codenotify: Notifying subscribers in CODENOTIFY files for diff 152ca0e...f8e2612.
|
c45a1af
to
66ca287
Compare
97a8d66
to
a6edf49
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks gorgeous.
I wrote up some feedback on the interface here: https://www.notion.so/metabase/Question-Model-picker-UI-feedback-28191f6f34cd42d89ba9cd5d28af20fe?pvs=4
For me the only major blocker is the selectedItem
issue mentioned below. I'm optimistically approving this PR
...end/src/metabase-enterprise/sandboxes/components/EditSandboxingModal/EditSandboxingModal.tsx
Show resolved
Hide resolved
frontend/src/metabase/common/components/EntityPicker/components/EntityPickerModal/TabsView.tsx
Outdated
Show resolved
Hide resolved
frontend/src/metabase/common/components/EntityPicker/components/EntityPickerModal/TabsView.tsx
Outdated
Show resolved
Hide resolved
frontend/src/metabase/common/components/QuestionPicker/components/QuestionPicker.tsx
Show resolved
Hide resolved
frontend/src/metabase/common/components/QuestionPicker/components/QuestionPicker.tsx
Outdated
Show resolved
Hide resolved
frontend/src/metabase/common/components/QuestionPicker/components/QuestionPicker.tsx
Outdated
Show resolved
Hide resolved
frontend/src/metabase/common/components/QuestionPicker/components/QuestionPicker.tsx
Outdated
Show resolved
Hide resolved
frontend/src/metabase/models/containers/FormModelPicker/FormModelPicker.tsx
Show resolved
Hide resolved
frontend/src/metabase/models/containers/ModelDetailPage/ModelDetailPage.tsx
Outdated
Show resolved
Hide resolved
frontend/src/metabase/parameters/components/ValuesSourceModal/ValuesSourceCardModal.tsx
Show resolved
Hide resolved
84c2993
to
e04b0b1
Compare
0df5169
to
41da831
Compare
41da831
to
eb99852
Compare
Closes #36376
Description
Adds a question/model picker using the entity picker.
Used by
Checklist