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: save filters in scheduled exports #5467

Merged
merged 1 commit into from
Oct 14, 2024
Merged

Conversation

kandl
Copy link
Contributor

@kandl kandl commented Oct 14, 2024

Do not override dashboard filters
in scheduled export with undefined value.

Remove unnecessary code.

risk: low
JIRA: F1-802


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

Do not override dashboard filters
in scheduled export with `undefined` value.

Remove unnecessary code.

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

gooddata bot commented Oct 14, 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/2589/ finished with 'SUCCESS' status.

@gooddata
Copy link

gooddata bot commented Oct 14, 2024

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

Merged via the queue into release with commit 73b97ad Oct 14, 2024
3 checks passed
@hackerstanislav hackerstanislav deleted the mkn-f1-802-release branch October 14, 2024 13:35
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.

2 participants