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.

Sign up for GitHub

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

Jump to bottom

CatalogDBAdaptor and CatalogManager with QueryOptions #28

Closed
j-coll opened this issue Dec 8, 2014 · 0 comments
Closed

CatalogDBAdaptor and CatalogManager with QueryOptions #28

j-coll opened this issue Dec 8, 2014 · 0 comments
Assignees
@j-coll
Labels
feature
Milestone
v0.4.0

Comments

@j-coll
Copy link
Member

j-coll commented Dec 8, 2014

This classes should accept a QueryOptions param with "exclude" or "include" with a List of fields path.

This fields can be relative to the returning element or absolute to the user. e.g:

getStudy( <ID>, QueryOptions{ include: ["alias"]} )

vs

getStudy( <ID>, QueryOptions{ include: ["projects.studies.alias"]} )

To unify the fields name between all the methods, it's recommended to use an absolute field path
@j-coll j-coll added the feature label Dec 8, 2014
@j-coll j-coll self-assigned this Dec 8, 2014
@j-coll j-coll added this to the v0.4.0 milestone Dec 8, 2014
@j-coll j-coll closed this as completed in a48dd55 Feb 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@j-coll j-coll

Labels
feature
Projects
None yet
Milestone
v0.4.0
Development

No branches or pull requests
1 participant
@j-coll