Skip to content
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

serialize result_metadata for any card #41131

Merged
merged 2 commits into from Apr 9, 2024
Merged

Conversation

piranha
Copy link
Contributor

@piranha piranha commented Apr 8, 2024

We need this for every card, since when a normal question depends on a native question - their result_metadata won't be updated when opening a dashboard, which leads to front-end not understanding how to connect the parameters, and errors during queries.

We need this for every card, since when a normal question depends on a native question - their result_metadata won't
be updated when opening a dashboard, which leads to front-end not understanding how to connect the parameters, and
errors during queries.
@piranha piranha added backport Automatically create PR on current release branch on merge .Team/BackendComponents also known as BEC labels Apr 8, 2024
@piranha piranha self-assigned this Apr 8, 2024
@piranha piranha requested a review from camsaul as a code owner April 8, 2024 13:59
Copy link

replay-io bot commented Apr 8, 2024

Status In Progress ↗︎ 53 / 54
Commit ccff67d
Results
⚠️ 15 Flaky
2390 Passed

@piranha piranha requested a review from a team April 8, 2024 14:34
Copy link
Member

@tsmacdonald tsmacdonald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 to tightening the test, but otherwise looks like it does what it says on the tin ⛵

@piranha piranha merged commit 1504b86 into master Apr 9, 2024
107 checks passed
@piranha piranha deleted the extract-result-metadata branch April 9, 2024 06:50
Copy link

github-actions bot commented Apr 9, 2024

@piranha Did you forget to add a milestone to the issue for this PR? When and where should I add a milestone?

@piranha
Copy link
Contributor Author

piranha commented Apr 9, 2024

@metabase-bot backport release-x.48.x

Copy link

github-actions bot commented Apr 9, 2024

@piranha something went wrong while creating a backport [Logs]

@piranha piranha changed the title extract result metadata serialize result_metadata for any card Apr 9, 2024
@piranha piranha added this to the 0.48.11 milestone Apr 9, 2024
piranha added a commit that referenced this pull request Apr 9, 2024
Co-authored-by: Alexander Solovyov <alexander@solovyov.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Automatically create PR on current release branch on merge .Team/BackendComponents also known as BEC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants