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

Support populate queries #16

Open
jkeczan opened this issue Oct 14, 2015 · 6 comments
Open

Support populate queries #16

jkeczan opened this issue Oct 14, 2015 · 6 comments

Comments

@jkeczan
Copy link

jkeczan commented Oct 14, 2015

It appears that the adapter does not support the populate functionality of SailsJS. This feature would be extremely handy. All we would need to do is implement populate inside the adapter.

@devinivy
Copy link
Collaborator

It does not currently support populates. I'll mark this as the official feature request. Would definitely take a PR.

@devinivy devinivy changed the title Support SailJS Populate Support populate queries Oct 14, 2015
@muditjuneja
Copy link

populate is working in case of collections.
But when you try to populate other model it works for some models but do not work for other models.

Is it a bug ?

@jkeczan
Copy link
Author

jkeczan commented Jan 28, 2016

It currently doesn't have the functionality to perform the populate on other

@jkeczan jkeczan closed this as completed Jan 28, 2016
@jkeczan jkeczan reopened this Jan 28, 2016
@jkeczan
Copy link
Author

jkeczan commented Jan 28, 2016

Model. The feature hasn't been written yet

@taylorjames
Copy link

Has there been any progress with the populate functionality? @jkeczan did you come up with a solution?

If not I will try and come up with a PR

@jkeczan
Copy link
Author

jkeczan commented Jun 8, 2016

I haven't. I started working on it, but my company pulled me from the project so I never had the opportunity to finish it......I would welcome a PR if you have time

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

No branches or pull requests

4 participants