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

Restoring a bookmarked entity (dashboard/question/metric) leads to application crash #44224

Closed
romeovs opened this issue Jun 14, 2024 · 4 comments · Fixed by #44414
Closed

Restoring a bookmarked entity (dashboard/question/metric) leads to application crash #44224

romeovs opened this issue Jun 14, 2024 · 4 comments · Fixed by #44414
Assignees
Labels
.Frontend Organization/ Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness .Regression/master Regression that is only present on master, or bug in new upcoming feature .Team/AdminWebapp Admin and Webapp team Type:Bug Product defects
Milestone

Comments

@romeovs
Copy link
Contributor

romeovs commented Jun 14, 2024

Describe the bug

Restoring a bookmarked metric results in an application crash (js).

Metric.A.Metabase.-.14.June.2024.mp4

To Reproduce

  1. Navigate to a metric
  2. Bookmark the metric
  3. Move the metric to trash
  4. Restore the metric

Expected behavior

No response

Logs

No response

Information about your Metabase installation

{
  "browser-info": {
    "language": "en-US",
    "platform": "MacIntel",
    "userAgent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:126.0) Gecko/20100101 Firefox/126.0",
    "vendor": ""
  },
  "system-info": {
    "file.encoding": "UTF-8",
    "java.runtime.name": "OpenJDK Runtime Environment",
    "java.runtime.version": "11.0.20.1+1",
    "java.vendor": "Eclipse Adoptium",
    "java.vendor.url": "https://adoptium.net/",
    "java.version": "11.0.20.1",
    "java.vm.name": "OpenJDK 64-Bit Server VM",
    "java.vm.version": "11.0.20.1+1",
    "os.name": "Mac OS X",
    "os.version": "14.2.1",
    "user.language": "en",
    "user.timezone": "Europe/Amsterdam"
  },
  "metabase-info": {
    "databases": [
      "h2"
    ],
    "hosting-env": "unknown",
    "application-database": "h2",
    "application-database-details": {
      "database": {
        "name": "H2",
        "version": "2.1.214 (2022-06-13)"
      },
      "jdbc-driver": {
        "name": "H2 JDBC Driver",
        "version": "2.1.214 (2022-06-13)"
      }
    },
    "run-mode": "dev",
    "version": {
      "date": "2024-06-12",
      "src_hash": "7a87e1a1173927a489fc70b6fc3e9bded123a326",
      "tag": "v1.2.0-SNAPSHOT",
      "hash": "3afe482"
    },
    "settings": {
      "report-timezone": null
    }
  }
}

Severity

P2

Additional context

No response

@romeovs romeovs added Type:Bug Product defects .Needs Triage labels Jun 14, 2024
@ranquild ranquild added Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness .Frontend Querying/Metrics v2 .Regression/master Regression that is only present on master, or bug in new upcoming feature .Team/QueryingComponents and removed Administration/Metrics & Segments .Needs Triage labels Jun 14, 2024
@NevRA
Copy link
Contributor

NevRA commented Jun 18, 2024

@dpsutton is not related to the Metricsv2, so re-assigned AdminWebApp

@npfitz npfitz self-assigned this Jun 18, 2024
@perivamsi perivamsi changed the title Restoring a bookmarked metric leads to application crash Restoring a bookmarked entity (dashboard/question/metric) leads to application crash Jun 18, 2024
@perivamsi
Copy link
Contributor

Screen.Recording.2024-06-18.at.9.44.59.AM.mov

@ranquild
Copy link
Contributor

@npfitz can we close this issue since your PR is merged?

@npfitz npfitz closed this as completed Jun 20, 2024
@npfitz
Copy link
Contributor

npfitz commented Jun 20, 2024

Yup. PR is merged in release branch. PR to master is up and pending.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.Frontend Organization/ Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness .Regression/master Regression that is only present on master, or bug in new upcoming feature .Team/AdminWebapp Admin and Webapp team Type:Bug Product defects
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants