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
feat: activate not-exists for link prop #992
Conversation
DEV-1966 Activate not-exists for link
The not exists in the advanced search was commented out for the linking property case. After investigation & fixing a potential bug on the api side, this should be activated |
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.
LGTM, minor comments
@@ -323,45 +323,45 @@ describe('SpecifyPropertyValueComponent', () => { | |||
).toEqual(8); | |||
}); | |||
|
|||
// it('should set the correct comparison operators for a linking property type (on top level)', () => { |
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.
Is this dead code now? I'd delete it if it's not used.
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.
No, I uncommented it so the test is valid for the added notexists case
@@ -2246,54 +2246,54 @@ OFFSET 0 | |||
).toHaveBeenCalledTimes(1); | |||
}); | |||
|
|||
// it('Gravsearch query string with no existing LINK property values', () => { |
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.
Same as above
resolves DEV-1966