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 cross-filtering attribute names #5346

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix cross-filtering attribute names #5346

wants to merge 2 commits into from

Conversation

scavnickyj
Copy link
Contributor


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

We want to respect "column" name of visualization
when creating virtual filters.
Custom column names will also work this way.

JIRA: F1-524
risk: low
@scavnickyj scavnickyj added the do not merge Do not merge this yet label Sep 12, 2024
@gooddata
Copy link

gooddata bot commented Sep 12, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Do not merge this yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant