-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
isBase64Url() #1212
Comments
PR welcome! |
I like to look into this one |
Seems that using a simple 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 hapijs/joi#1746 |
@profnandaa can i do a pr? |
please do it.
/na
On Sat, Dec 28, 2019 at 5:41 PM Rubin Bhandari ***@***.***> 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
<#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 #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 :) |
Feature request.
The text was updated successfully, but these errors were encountered: