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

need massCheck function capability #96

Closed
wsugiri opened this issue Feb 8, 2015 · 9 comments
Closed

need massCheck function capability #96

wsugiri opened this issue Feb 8, 2015 · 9 comments

Comments

@wsugiri
Copy link

wsugiri commented Feb 8, 2015

is there any function like massCheck in new version like old version?

@casperin
Copy link
Owner

casperin commented Feb 8, 2015

To be honest I don't quite remember what exactly it did, but I imagine it did something like this: https://github.com/casperin/nod#manually-checking-validity

myNod.areAll('valid');

Will that work for you?

@wsugiri
Copy link
Author

wsugiri commented Feb 10, 2015

thank you for the function, but not exactly same with massCheck.
in old version massCheck, we can get both the result and update the screen to (give visual error).
function myNod.areAll('valid') only get the result valid or not.

by the way thank you for the feature.

@wsugiri wsugiri closed this as completed Feb 10, 2015
@casperin
Copy link
Owner

Aaaah right.. it would basically trigger a change on every element, forcing nod to run its checks.

I guess there is no equivalent to that, sorry.

@wsugiri
Copy link
Author

wsugiri commented Feb 11, 2015

never mind, my team already get the work around by modify the checkers and add equivalent capability.

thanks.

@casperin
Copy link
Owner

Glad to hear that :) And thanks for using my little library

@wsugiri
Copy link
Author

wsugiri commented Feb 12, 2015

yes.., your library save me a lot of time to manage validation.
for trigger visual element, we add little code to make massChange capability

    function massCheck () {
        checkers.forEach(function(checker) {
            checker.performCheck();
        })
    }

then this feature available in the latest version, my be the next release you will add this capability. thanks...

@casperin
Copy link
Owner

Hmm, yeah. I might do that. Why not. Different name though I think. checkAll() or something. I'll keep it in mind :)

@casperin
Copy link
Owner

https://github.com/casperin/nod#force-nod-to-perform-a-check :)

I pretty much just copied your implementation. Just added a bit of extra functionaility:
https://github.com/casperin/nod/blob/master/nod.js#L360

@wsugiri
Copy link
Author

wsugiri commented Feb 14, 2015

That's great, thank you for the implementation and fast response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants