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

enhancement: Add support for negating conditions in check_fields #2514

Merged
merged 3 commits into from May 4, 2020

Conversation

bruceg
Copy link
Member

@bruceg bruceg commented Apr 30, 2020

This applies to the filter and swimlanes transforms.

This negation may apply to any check_fields condition.

Signed-off-by: Bruce Guenter bruce@timber.io

Closes #2446

This applies to the filter and swimlanes transforms.

This negation may apply to any check_fields condition.

Signed-off-by: Bruce Guenter <bruce@timber.io>
@bruceg bruceg added type: enhancement A value-adding code change that enhances its existing functionality. transform: route Anything `route` transform related transform: filter Anything `filter` transform related labels Apr 30, 2020
@bruceg bruceg self-assigned this Apr 30, 2020
[<%= namespace %>."`[field-name]`.not_`[condition]`"]
type = "any"
examples = [
{ "message.not_contains" = "some phrase to ignore" }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add a few more examples here?

Copy link
Contributor

@binarylogic binarylogic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docs look good

Bruce Guenter added 2 commits May 4, 2020 10:08
Signed-off-by: Bruce Guenter <bruce@timber.io>
Signed-off-by: Bruce Guenter <bruce@timber.io>
Copy link
Member

@lukesteensen lukesteensen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! This is very neat.

@bruceg bruceg merged commit ec2ec16 into master May 4, 2020
@bruceg bruceg deleted the condition-negation branch May 4, 2020 22:26
@pySilver
Copy link

Great! :) This should go to docs too 👍

mengesb pushed a commit to jacobbraaten/vector that referenced this pull request Dec 9, 2020
…tordotdev#2514)

* enhancement: Add support for negating conditions in check_fields

This applies to the filter and swimlanes transforms.

This negation may apply to any check_fields condition.

Signed-off-by: Bruce Guenter <bruce@timber.io>
Signed-off-by: Brian Menges <brian.menges@anaplan.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
transform: filter Anything `filter` transform related transform: route Anything `route` transform related type: enhancement A value-adding code change that enhances its existing functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

How to specify "not.contains" transform?
4 participants