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

BE: MLv2 Add support for segment and metric in legacyFieldRef #36699

Closed
Tracked by #30513
uladzimirdev opened this issue Dec 12, 2023 · 0 comments · Fixed by #36718
Closed
Tracked by #30513

BE: MLv2 Add support for segment and metric in legacyFieldRef #36699

uladzimirdev opened this issue Dec 12, 2023 · 0 comments · Fixed by #36718
Assignees
Labels
.metabase-lib Label for tracking all issues related to the shared CLJC metabase-lib .Task Not a part of any Epic, used by the Task Issue Template .Team/QueryProcessor :hammer_and_wrench:
Milestone

Comments

@uladzimirdev
Copy link
Contributor

legacyFieldRef should accept segment and metric together with prev behaviour (column).

before MLv2 segments and metrics didn't support options, so the expected result should be a structure like this:

['segment', id]

@uladzimirdev uladzimirdev added the .Task Not a part of any Epic, used by the Task Issue Template label Dec 12, 2023
@metamben metamben self-assigned this Dec 12, 2023
@darksciencebase darksciencebase added .Team/QueryProcessor :hammer_and_wrench: .metabase-lib Label for tracking all issues related to the shared CLJC metabase-lib labels Dec 12, 2023
@uladzimirdev uladzimirdev changed the title MLv2 Add support for segment and metric in legacyFieldRef BE: MLv2 Add support for segment and metric in legacyFieldRef Dec 13, 2023
@metamben metamben added this to the 0.49 milestone Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.metabase-lib Label for tracking all issues related to the shared CLJC metabase-lib .Task Not a part of any Epic, used by the Task Issue Template .Team/QueryProcessor :hammer_and_wrench:
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants