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

fix: add npm downloads total #17

Merged
merged 2 commits into from
Jul 19, 2018
Merged

fix: add npm downloads total #17

merged 2 commits into from
Jul 19, 2018

Conversation

tunnckoCore
Copy link
Contributor

@tunnckoCore tunnckoCore commented Jul 19, 2018

resolves #13

Huh. That was fast and fun! 🎉

That's a service, that's a library, hooorey!

Shields sucks, for years. I started such service with koa back in 2014 if i remember correctly. But never got the time to finish it "officially" and release it.

It was times that there was mostly ES5 and no Babel or such :D
It was times, that Shields "server.js" was around 2000-3000 lines of code...
and HELL they still add to that file?! Currently it is 7900+, wtf.

Signed-off-by: Charlike Mike Reagent <olsten.larck@gmail.com>
@vercel
Copy link

vercel bot commented Jul 19, 2018

Since this Pull Request originated from a forked repository, Now cannot deploy it as there are potential security risks.

If you are a collaborator on this repository, consider making this Pull Request from a branch on the same repository instead of a fork.

Signed-off-by: Charlike Mike Reagent <olsten.larck@gmail.com>
@amio
Copy link
Member

amio commented Jul 19, 2018

Glad to have you here! And thanks!

@amio amio merged commit f624f4d into badgen:master Jul 19, 2018
@tunnckoCore
Copy link
Contributor Author

btw, downloads/day seems bugged? it shows 0/day for all packages?

@tunnckoCore
Copy link
Contributor Author

That's before my merged updates.

@amio
Copy link
Member

amio commented Jul 21, 2018

I noticed that, while the api did returns 0, I have no idea & leave that bug there.

Wait a sec, there's a https://api.npmjs.org/downloads/point/2018-07-19/express 😄

@amio
Copy link
Member

amio commented Jul 21, 2018

Ahha, you already done that fix 😆

@tunnckoCore
Copy link
Contributor Author

Oh cool. Didn't noticed that endpoint. Will PR to change to it instead of /range.

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

Successfully merging this pull request may close these issues.

Support for npm downloads total
2 participants