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

Create a list method for the wep app to return all the packages #179

Closed
wants to merge 4 commits into from

Conversation

yannickglt
Copy link

This method would allow to get all the packages available on the local registry.

@rlidwka
Copy link
Owner

rlidwka commented Dec 12, 2014

What is it used for, and what does it have to do with web app?

@yannickglt
Copy link
Author

The purpose of this method is to be used with an external system. In my case, the method will be requested by a yeoman generator to suggest the user to use a package available on our internal registry to quickly bootstrap new projects.
I moved the method from the wep app to the api app. It makes more sense as it is an equivalent of the index page of the web app but for the API.

@rlidwka
Copy link
Owner

rlidwka commented Dec 12, 2014

Maybe /-/all?local would be better? We already have that for search, and "local" option would be easy to add.

@yannickglt
Copy link
Author

Ok good idea. That is done on my last commit.

@rlidwka
Copy link
Owner

rlidwka commented Dec 14, 2014

Squashed and merged in ea50f3d, thanks!

@rlidwka rlidwka closed this Dec 14, 2014
rmg pushed a commit to strongloop-forks/sinopia that referenced this pull request Jun 1, 2017
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.

2 participants