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 null values being applied to filters with click behavior custom URL links #40023
Fix null values being applied to filters with click behavior custom URL links #40023
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The fix looks good! Could we please add a test for it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, please update the field reference before merging 🙏
@JesseSDevaney Did you forget to add a milestone to the issue for this PR? When and where should I add a milestone? |
…RL links (#40023) * do not pass null values to link url * update null check to include query builder nulls replaced with "(empty") * add test to verify behavior * replace magic string with constant * fix breakout in cypress spec
…RL links (#40023) (#40323) * do not pass null values to link url * update null check to include query builder nulls replaced with "(empty") * add test to verify behavior * replace magic string with constant * fix breakout in cypress spec Co-authored-by: Jesse Devaney <22608765+JesseSDevaney@users.noreply.github.com>
…RL links (#40023) * do not pass null values to link url * update null check to include query builder nulls replaced with "(empty") * add test to verify behavior * replace magic string with constant * fix breakout in cypress spec
Before
2024-03-12.13-35-14.mp4
After
2024-03-12.13-36-52.mp4