search

arve0 / npm-download-size

analyze size of npm packages
https://arve0.github.io/npm-download-size
33 stars 4 forks source link

Doesn't work for scoped packages #2

Closed smiller171 closed 6 years ago

smiller171 commented 6 years ago

If I try to check on a scoped package (@nuxtjs/vuetify) the package is incorrectly reported as not existing.

arve0 commented 6 years ago

Thanks for reporting. Away for about a week, will check when I’m back.

fre. 23. mar. 2018 kl. 20:31 skrev Scott Miller notifications@github.com:

If I try to check on a scoped package (@nuxtjs/vuetify) the package is incorrectly reported as not existing.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/arve0/npm-download-size/issues/2, or mute the thread https://github.com/notifications/unsubscribe-auth/AElnGVj5OqNS2Yptz_ROcGUFE7VmTnzPks5thU14gaJpZM4S5Q7Q .

smiller171 commented 6 years ago

Turns out you need to replace the / with %2F (@nuxtjs%2Fvuetify)

smiller171 commented 6 years ago

@arve0 did you take a look at this PR?

arve0 commented 6 years ago

I’m still on vacation and have no computer with me.

søn. 1. apr. 2018 kl. 05:50 skrev Scott Miller notifications@github.com:

@arve0 https://github.com/arve0 did you take a look at this PR?

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/arve0/npm-download-size/issues/2#issuecomment-377741719, or mute the thread https://github.com/notifications/unsubscribe-auth/AElnGc39V3APWQW6sV-CgxQtIGWJRBDDks5tkE6DgaJpZM4S5Q7Q .

arve0 commented 6 years ago

The PR did not solve the issue. One would probably need to take a look at the server side.

arve0 commented 6 years ago

Was related to issue https://github.com/npm/registry/issues/34. Fix is on the way.