-
Notifications
You must be signed in to change notification settings - Fork 102
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
Added filter for empty rows in Define New Filter dialog #7629
Added filter for empty rows in Define New Filter dialog #7629
Conversation
That's an interesting way to do it - and seems to work fine. I thought we would just enable the |
@rdstern I agree to the request above and I have actioned it accordingly. |
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.
@shadrackkibet this is great. Not the way I was thinking, but this way seems better. And excellent that you are back to R-Instat, even though no longer with INNODEMS. I hope @lloyddewit can approve, so you can proceed to the useful ADD facility after this is merged!
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.
@shadrackkibet Tidy fix, thanks, happy to approve.
I just fixed a small indentation issue.
fixes #7617
@rdstern I added an option "isEmpty" into the dropdown. When you select this option it allows you to click Add Condition then you can select only empty rows. What do you think?