Skip to content
This repository has been archived by the owner on Jul 11, 2021. It is now read-only.

Document "TODO" for storage configuration parameter in the packages section #16

Closed
FrontierPsychiatrist opened this issue Dec 20, 2018 · 5 comments

Comments

@FrontierPsychiatrist
Copy link

In https://github.com/verdaccio/website/blob/master/docs/packages.md there is still a TODO in the bottom table for parameters.

I set up a Verdaccio registry recently and thought I had to switch this to true because I wanted some scoped packages to be stored. But setting this to true caused npm publish to fail (404 error).

So it would be nice to add documentation for this parameter (or remove it as long as it's not described what it does).

@juanpicado
Copy link
Member

hi @FrontierPsychiatrist which parameter? Could you share an example?

@FrontierPsychiatrist
Copy link
Author

Hey, sure!

packages:
  '@somescope/*':
    access: $all
    publish: $authenticated
    storage: true

This boolean is documented with TODO on the page I linked, in the table at the bottom.

@juanpicado
Copy link
Member

juanpicado commented Dec 21, 2018

I set up a Verdaccio registry recently and thought I had to switch this to true because I wanted some scoped packages to be stored. But setting this to true caused npm publish to fail (404 error).

I see, the storage was never documented for a couple of reasons. But I guess you can read here reasons verdaccio/verdaccio#965 (comment)

If you are using verdaccio@3.x . I'd not use it yet. The tarballs are cached by default, not need to add that parameter.

@juanpicado
Copy link
Member

If you want to PR the right behavior would be great.

@juanpicado
Copy link
Member

Updated.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants