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

When updating a filter value in chill mode, the order of the columns is messed up #42049

Open
albertoperdomo opened this issue Apr 30, 2024 · 3 comments
Labels
Priority:P2 Average run of the mill bug Querying/GUI Query builder catch-all, including simple mode .Team/QueryingComponents Type:Bug Product defects

Comments

@albertoperdomo
Copy link
Member

albertoperdomo commented Apr 30, 2024

Describe the bug

When updating a filter value in chill mode, the order of the columns is messed up.

To Reproduce

  1. Go to stats https://stats.metabase.com/question/4859
  2. Click on filters on the top bar
  3. Click on the filter Created At is in the previous week, starting 16 weeks ago
  4. Change the value of starting from, from 16 to 17 weeks
  5. Click on update filter
  6. The order of the columns is not preserved, notice the 2nd column is now different.

Slack thread w/ Loom https://metaboat.slack.com/archives/C01LQQ2UW03/p1714409454229319

blocked by RFC

there is viz settings -> cols mismatch as cols's format of the field_ref is ['field', 'created_at'] for some columns and viz settings use ['field', 34] where 34 is created_at id, the issue can't be addressed in the current state of field_refs, so we expect it to be fixable after RFC gets implemented

Expected behavior

The order of the columns is preserved

Logs

No response

Information about your Metabase installation

- Stats Built on 2024-04-30 Hash: 389886d

Severity

P2

Additional context

No response

@zbodi74 zbodi74 added Priority:P2 Average run of the mill bug Querying/GUI Query builder catch-all, including simple mode labels May 2, 2024
@zbodi74
Copy link

zbodi74 commented May 2, 2024

This has also been reported by a customer in v1.49.7.

@albertoperdomo
Copy link
Member Author

The order of columns is also messed up jas well when sorting by a specific column.

@uladzimirdev
Copy link
Contributor

uladzimirdev commented May 6, 2024

@albertoperdomo you're right, this problem is related to the latest rework to MLv2, I'm trying to understand where we should fix it - FE or BE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:P2 Average run of the mill bug Querying/GUI Query builder catch-all, including simple mode .Team/QueryingComponents Type:Bug Product defects
Projects
None yet
Development

No branches or pull requests

3 participants