-
-
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
[proposal] add CDN option in readme #1507
Comments
Many people are using unpkg to load JS libraries. I really think it's a great idea to add instructions for that purpose specially that |
Sounds good to me, let's do that. Thanks! |
Busy working on some work, but here's the address if anyone has time to update the readme. If I can, I'll get to it before end of day. https://unpkg.com/validator.js@latest/dist/validator.min.js Edit: Of course I say that, but didn't test that min.js worked for me. Derp. |
In theory, the below should work?
|
Sure @Brutalbeard. Please if you get sometime later in the day, you can add it. |
@ezkemboi, @profnandaa, @luiscobits or @tux-tn, can anyone just double check that they have no issue bringing this into a basic HTML page? I'll have a pull request for the readme ready if you give the go ahead. |
Hey all. Just bumping on this. Haven't used unpkg.com before. Thinking there may be some other dependency to add? @ezkemboi, @profnandaa, @luiscobits, @tux-tn |
Thanks @Brutalbeard! No, Unpkg do not need anything else, only the link that you pointed out before. |
PR on its way! |
Just a head's up: I think you probably want: |
@Brutalbeard -- sure, looking forward to your PR. |
Hi, I want to proposal to add cdn instructions in the readme, we can use https://unpkg.com/
In that way, we will have a cdn method to add validator.js in client-side.
Can I make a PR?
The text was updated successfully, but these errors were encountered: