Skip to content
This repository has been archived by the owner on May 8, 2020. It is now read-only.

Implement common query interface #13

Open
lutzmor opened this issue Apr 18, 2017 · 0 comments
Open

Implement common query interface #13

lutzmor opened this issue Apr 18, 2017 · 0 comments

Comments

@lutzmor
Copy link
Contributor

lutzmor commented Apr 18, 2017

Currently the Query type passed to store.findMany is undefined. It needs to be a common interface that makes sense in any store, not just the database store.

  • It needs to be mappable to typeorm's FindOptions
  • It must be transportable over json
  • Considerations for security must be taken (probably avoiding all join capabilities to avoid entity permission leak)
  • Consider severely limiting to force controllers to do the querying if security concerns are too great

Another comment by zakhenry

  • Consider graphql, it makes sense across multiple database scheme & transports. Tough part may be data hydration
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

1 participant