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

fix: call onError when insight does not exist #5055

Merged
merged 1 commit into from
Jun 25, 2024
Merged

Conversation

kandl
Copy link
Contributor

@kandl kandl commented Jun 25, 2024

Fix InsightView to call onError callback, when insight does not exist.

risk: low
JIRA: F1-442


Important

Please, don't forget to run rush change for the commits that introduce new features 🙏


Refer to documentation to see how to run checks and tests in the pull request. This is the list of the most used commands:

extended test - backstop
extended test - tiger-cypress - integrated
extended test - tiger-cypress - isolated
extended test - tiger-cypress - record

Fix InsightView to call onError callback, when insight does not exist.

risk: low
JIRA: F1-442
@kandl kandl added ready to review PR is ready to final code review publish pre-release labels Jun 25, 2024
@gooddata
Copy link

gooddata bot commented Jun 25, 2024

Job gooddata-ui-sdk-cypress-tiger-isolated-zuul-pipeline at https://checklist.intgdc.com/job/gooddata-ui-sdk/job/gooddata-ui-sdk-cypress-tiger-isolated-zuul-pipeline/2098/ finished with 'SUCCESS' status.

@gooddata
Copy link

gooddata bot commented Jun 25, 2024

Job gooddata-ui-sdk-unit-tests at https://checklist.intgdc.com/job/gooddata-ui-sdk/job/gooddata-ui-sdk-unit-tests/20960/ finished with 'SUCCESS' status.

Merged via the queue into master with commit 4c42f97 Jun 25, 2024
4 of 6 checks passed
@scavnickyj scavnickyj deleted the mkn-f1-442 branch June 25, 2024 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
publish pre-release ready to review PR is ready to final code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants