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

Changing existing filters in Simple GUI will replace other existing filters #24664

Closed
jbarnesdafg opened this issue Aug 8, 2022 · 0 comments · Fixed by #24677
Closed

Changing existing filters in Simple GUI will replace other existing filters #24664

jbarnesdafg opened this issue Aug 8, 2022 · 0 comments · Fixed by #24677
Assignees
Labels
.Correctness .Frontend Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness Querying/GUI Query builder catch-all, including simple mode Querying/Parameters & Variables Filter widgets, field filters, variables etc. .Regression Bugs that were previously fixed and/or bugs unintentionally shipped with new features. .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects
Milestone

Comments

@jbarnesdafg
Copy link

Describe the bug
Filter chips in questions will overwrite first filter rather than update. This is an issue on all questions.

Screenshots
image
**
image

Information about your Metabase Installation:
{
"browser-info": {
"language": "en-US",
"platform": "Win32",
"userAgent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/103.0.0.0 Safari/537.36",
"vendor": "Google Inc."
},
"system-info": {
"file.encoding": "UTF-8",
"java.runtime.name": "OpenJDK Runtime Environment",
"java.runtime.version": "11.0.14+9-Ubuntu-0ubuntu2.16.04",
"java.vendor": "Ubuntu",
"java.vendor.url": "https://ubuntu.com/",
"java.version": "11.0.14",
"java.vm.name": "OpenJDK 64-Bit Server VM",
"java.vm.version": "11.0.14+9-Ubuntu-0ubuntu2.16.04",
"os.name": "Linux",
"os.version": "4.4.0-101-generic",
"user.language": "en",
"user.timezone": "US/Eastern"
},
"metabase-info": {
"databases": [
"sqlserver",
"db2",
"postgres"
],
"hosting-env": "unknown",
"application-database": "postgres",
"application-database-details": {
"database": {
"name": "PostgreSQL",
"version": "9.5.25"
},
"jdbc-driver": {
"name": "PostgreSQL JDBC Driver",
"version": "42.3.5"
}
},
"run-mode": "prod",
"version": {
"date": "2022-08-05",
"tag": "v1.44.1-SNAPSHOT",
"branch": "release-x.44.x",
"hash": "1c661ef"
},
"settings": {
"report-timezone": "US/Eastern"
}
}
}

Severity
The side bar filter and editor filter do not have this issue and are the current work around.

@alxnddr alxnddr self-assigned this Aug 8, 2022
@alxnddr alxnddr added the .Regression Bugs that were previously fixed and/or bugs unintentionally shipped with new features. label Aug 8, 2022
@flamber flamber added Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness .Frontend Querying/GUI Query builder catch-all, including simple mode Querying/Parameters & Variables Filter widgets, field filters, variables etc. .Correctness and removed .Needs Triage labels Aug 9, 2022
@flamber flamber changed the title Filter Chips on Questions Overwriting Other Filters Changing existing filters in Simple GUI will replace other existing filters Aug 9, 2022
@flamber flamber added this to the 0.44.1 milestone Aug 9, 2022
@alxnddr alxnddr added the .Reproduced Issues reproduced in test (usually Cypress) label Aug 9, 2022
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
.Correctness .Frontend Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness Querying/GUI Query builder catch-all, including simple mode Querying/Parameters & Variables Filter widgets, field filters, variables etc. .Regression Bugs that were previously fixed and/or bugs unintentionally shipped with new features. .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants