-
Notifications
You must be signed in to change notification settings - Fork 100
fix: enable inq filters for referancesMany relations #783
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
base: master
Are you sure you want to change the base?
Conversation
Pull Request Test Coverage Report for Build 17603913986Details
💛 - Coveralls |
84723f3 to
9d6077c
Compare
9d6077c to
77eccac
Compare
Signed-off-by: Muhammad Aaqil <aaqilcs102@gmail.com>
77eccac to
72397d5
Compare
| SQLConnector.prototype.buildWhere = function(model, where, options) { | ||
| let relationType = ''; | ||
| let relationKeyFrom = ''; | ||
| if (options && options['model'] && options['model']['definition']) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ideally this should be part of datasource juggler repo as thats where we implement relations.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, @samarpanB. The relations are defined in the datasource-juggler, but here I am trying to modify the SQL queries to enable the inq filter for referencesMany relations.
inq filter fails in referencesMany relations. This PR fixes that.
Related PR@loopback-next
Checklist
npm testpasses on your machine