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

Text filter over a custom column in simple embedding expects a number #25473

Closed
paoliniluis opened this issue Sep 17, 2022 · 1 comment · Fixed by #27119
Closed

Text filter over a custom column in simple embedding expects a number #25473

paoliniluis opened this issue Sep 17, 2022 · 1 comment · Fixed by #27119
Assignees
Labels
Embedding/Public Simple public iframe embeds Embedding/Static Static embedding, previously known as signed embedding .Frontend Priority:P2 Average run of the mill bug Querying/Notebook/Custom Column Querying/Parameters & Variables Filter widgets, field filters, variables etc. .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects
Milestone

Comments

@paoliniluis
Copy link
Contributor

Describe the bug
When there's a text filter on a custom column in signed embedding, it will request for a number instead of a string

Logs
NA, seems like a FE error

To Reproduce

  1. enable embedding
  2. new->question->people>custom colum-> concat([Email], [Name]), save it with any name
  3. add question to dashboard
  4. add text filters (text contains, text does not contain, text starts with, text ends with)
  5. click to embed it, enable the filters as editable
  6. click on the filters to see all requesting a number

Expected behavior
Text filter should expect strings

Screenshots
image

Information about your Metabase Installation:

  • Your browser and the version: Brave latest
  • Your operating system: Pop 22.04
  • Your databases: Postgres 14
  • Metabase version: 1.44.2
  • Metabase hosting environment: Docker
  • Metabase internal database: Postgres

Severity
P2

Additional context
Workaround: use a view in the DB to build the custom column

@paoliniluis paoliniluis added Type:Bug Product defects Priority:P2 Average run of the mill bug Querying/Parameters & Variables Filter widgets, field filters, variables etc. Embedding/Static Static embedding, previously known as signed embedding labels Sep 17, 2022
@flamber
Copy link
Contributor

flamber commented Sep 17, 2022

Related to #22788

This was referenced Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Embedding/Public Simple public iframe embeds Embedding/Static Static embedding, previously known as signed embedding .Frontend Priority:P2 Average run of the mill bug Querying/Notebook/Custom Column Querying/Parameters & Variables Filter widgets, field filters, variables etc. .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects
Projects
Development

Successfully merging a pull request may close this issue.

6 participants