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

🤖 backported "Ensure :effective-type for columns from an aggregation in a card" #47942

Merged

Conversation

github-automation-metabase
Copy link
Contributor

#47936

Important

Manual conflict resolution is required.
Checkout the branch and run ./backport.sh script. Force push your changes after cherry-picking.

)

* Ensure :effective-type for columns from an aggregation in a card

Fixes #47184

* Synchronize effective-type with the base-type on override

When :base-type in the column metadata is overridden with the :base-type in the field ref, set it as :effective-type
too.  If :effective-type is also set in the field ref, it wins.
@metamben metamben force-pushed the backport-d8786ffc632f8781950539f219605a02b6e8e620 branch from 324fe6d to 96d46e9 Compare September 16, 2024 14:45
@metamben metamben merged commit e6f09d2 into release-x.50.x Sep 16, 2024
108 checks passed
@metamben metamben deleted the backport-d8786ffc632f8781950539f219605a02b6e8e620 branch September 16, 2024 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
was-backported apply this to PRs that are themselves backports
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants