Feature request.
PR welcome!
I like to look into this one
Seems that using a simple /^[A-Za-z0-9\-_]+$/regex for this is common.
I'm unaware of any other pattern to follow here.
@tmikaeld correct me if i am wrong but this does look like the issue. If it is, i will get working https://github.com/hapijs/joi/issues/1746
@profnandaa can i do a pr?
please do it.
/na
On Sat, Dec 28, 2019 at 5:41 PM Rubin Bhandari notifications@github.com
wrote:
@profnandaa https://github.com/profnandaa can i do a pr?
—
You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub
https://github.com/validatorjs/validator.js/issues/1212?email_source=notifications&email_token=AAB7ZEL2O3EAYCW2JTPBABLQ25QTJA5CNFSM4JYAUHP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHYLG2I#issuecomment-569422697,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB7ZEIA4QJILPWEMBPQDN3Q25QTJANCNFSM4JYAUHPQ
.>
Sent from a tiny device while on the move.
this Pr resolves this https://github.com/validatorjs/validator.js/pull/1223
@profnandaa I couldn't find a method with this name, has it been renamed or yet to be implemented?
would raise a pr if not already implemented
@mum-never-proud Doesn't seem to be implemented, the PR that's been merged seem to be empty. Since the ball has been dropped, feel free to make a new PR.
Sure.
cool, I have raised the PR :)