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

Unable to delete filter from saved question - Uncaught TypeError #28874

Closed
likeshumidity opened this issue Mar 2, 2023 · 0 comments · Fixed by #28945
Closed

Unable to delete filter from saved question - Uncaught TypeError #28874

likeshumidity opened this issue Mar 2, 2023 · 0 comments · Fixed by #28945
Assignees
Labels
Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects

Comments

@likeshumidity
Copy link
Contributor

Read this before creating a bug report

  • if any of the sections are incomplete, we might close the bug report since we won't be able to reproduce it
  • have you checked if the bug has already been reported? As we close duplicate reports, please search on the issue tracker if the problem has been reported before
  • use as many screenshots or videos as possible to describe what you're seeing, all visual representations speak for themselves

Describe the bug
Using Chrome in latest (stats), unable to delete any columns or summarize components. Receive a frontend error "Cannot read properties of null (reading '0')"

Each error fired when attempting to click an "X" next to any item in filter or summarize to the left:
image

To Reproduce
Steps to reproduce the behavior (if you can reproduce the bug using the Sample Database, we will find the issue faster):

  • attempt to edit question 5968 in Metabase stats

Expected behavior
Expect to see a permissions error or a more useful message about why they cannot be removed, or expect them to be removed.

Severity

  • makes question read-only

Additional context
Add any other context about the problem here.

@paoliniluis paoliniluis added Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness .Blocker and removed .Needs Triage labels Mar 3, 2023
@ranquild ranquild self-assigned this Mar 3, 2023
ranquild added a commit that referenced this issue Mar 6, 2023
@nemanjaglumac nemanjaglumac added the .Reproduced Issues reproduced in test (usually Cypress) label Jun 5, 2023
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
Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants