Add `onComplete` optional prop about react-verification-input HOT 4 CLOSED

Thank you for the kind words.

I like your proposal, because most of the time we're probably interested in complete values only and I've seen many people check for the length in the onChange handler.

You can go ahead and implement it if you want.

As for the naming, I can't think of anything else. So we can go with that, I guess.

from react-verification-input.

Hey, @andreaswilli!
Thank you for the reply! I added this to my doing list, and I'll try to work on that during this weekend.

Just so we can be on the same page: my initial idea was to check if the newInputVal.length is equal to the length. If true, check if the onComplete prop exists and then call it. What do you think about this approach?

if (RegExp(`^[${validChars}]{0,${length}}$`).test(newInputVal)) { 
   if (onChange) { 
     onChange?.(newInputVal); 
   }
   
   setLocalValue(newInputVal);
   
+   if (newInputVal.length === length) {
+     onComplete?.(newInputVal)
+   }
} 

Let me know if you think there is a better approach!

from react-verification-input.

Looks good to me 👍🏼

from react-verification-input.

Released v3.3.0

from react-verification-input.