Skip to content

Conversation

IvanBernatovic
Copy link

This PR is related to this ticket. I tested it locally for client-side data-tables component and everything worked fine and data is properly filtered by default.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 97.788% when pulling 0f82227 on IvanBernatovic:master into 3941eb9 on njleonzhang:master.

@njleonzhang
Copy link
Owner

@IvanBernatovic would you please help to write a unit test?

@IvanBernatovic
Copy link
Author

@njleonzhang Sure, I'll look into it and let you know more. I intend to work with this file which seems is related to checkboxes on client side, I'll also take the opportunity to test it with server side implementation and write unit test for that too.

I'll try to work on it this week. Do you have anything else in mind or do you have any advice/suggestion?

@njleonzhang
Copy link
Owner

@IvanBernatovic You are right. just add unit test in checkFilter.spec.js for client side and checkFilter.spec.js for server side.
I have reviewed your code and left a message here, please take a look.

@njleonzhang
Copy link
Owner

default checkbox filter is removed in v3.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants