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

Add regexp doc #143

Merged
merged 2 commits into from
May 30, 2017
Merged

Add regexp doc #143

merged 2 commits into from
May 30, 2017

Conversation

jannyHou
Copy link
Contributor

@jannyHou jannyHou commented May 29, 2017

Description

User complains '$regex' operator is not recognized. The reason is in loopback syntax it should be regexp, the connector converts it to a cloudant syntax $regex then send the query to database.

Related issues

connect to #20

Checklist

  • Improve the regexp doc

@@ -10,8 +10,15 @@ those typically used within Loopback.

### Regex Filtering

When filtering by `$regex` in Cloudant, you must provide at least one target field
that can be filtered with an equality operator.
LoopBack filter uses `regexp` as the regular expression field name, the connector converts it to a cloudant syntax name, which is `$regex`, then send the corresponding query to database. Therefore please provide `regexp` instead of `$regex` or `regex` in the filter of a LoopBack api, for example:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nitpick: sends the corresponding query and Therefore, please provide

@jannyHou
Copy link
Contributor Author

jannyHou commented May 30, 2017

@b-admike thanks changes applied. PTAL again :)

@jannyHou jannyHou merged commit ee2d940 into master May 30, 2017
@jannyHou jannyHou added apex and removed #review labels May 30, 2017
@jannyHou jannyHou added this to the Sprint 37 - Apex milestone May 31, 2017
@jannyHou jannyHou deleted the doc/fix-regexp branch June 8, 2017 17:29
kjdelisle pushed a commit that referenced this pull request Aug 22, 2017
 * Inherit couchdb functionalities (#163) (Sakib Hasan)
 * Add stalebot configuration (Kevin Delisle)
 * Recover & reuse couchdb2 tests (jannyHou)
 * Fix readme (ssh24)
 * Create Issue and PR Templates (#171) (Sakib Hasan)
 * Add CODEOWNER file (Diana Lau)
 * Recover manipulation.test.js (#159) (Janny)
 * Recover juggler tests (#158) (Janny)
 * Require init on mocha args (ssh24)
 * Do not strip _rev value on create (ssh24)
 * Fix docs on bulk replace op hooks (ssh24)
 * Fix update/updateAll function (ssh24)
 * Add cloudant specific bulkReplace function (ssh24)
 * Check error and result (#149) (Janny)
 * Fix updateAttributes function (ssh24)
 * Fix doc (#148) (Janny)
 * viewDocs (#133) (Janny)
 * Return back result count in updateAll (ssh24)
 * Fix database name typo on README (ssh24)
 * Add regexp doc (#143) (Janny)
 * Add proxy config test (#142) (Janny)
 * Allow users to spawn docker and run tests (ssh24)
 * test: use Cloudant 2.x based image for testing (Ryan Graham)
 * test: replace setup.sh with test.js (Ryan Graham)
 * Refactor functions in cloudant (ssh24)
 * Allow handling of ._rev on models (#123) (Kevin Delisle)
 * Allow travis to run against the latest code base (#138) (Sakib Hasan)
 * Add docker setup (#132) (Sakib Hasan)
 * Fix updateOrCreate (#136) (Sakib Hasan)
 * Fix typo (#135) (Janny)
 * cloudant.test: cleanup after test runs (Kevin Delisle)
 * Setup Travis with Docker Compose (Kevin Delisle)
 * Refactor doc (#116) (Janny)
 * reinstate bulk update (biniam)
 * add array prop update tests (biniam)
 * update docs with current revision (biniam)
 * Allow id property to be a number (#115) (Sakib Hasan)
 * autoupdate and automigrate fix (#109) (Janny)
 * update readme to doc async connect (biniam)
 * check cloudant db in config (biniam)
 * call driver asynchronously (biniam)
 * Fix sort query builder (#107) (Janny)
 * Recover maxrows.test.js (#91) (Janny)
 * Fix regexp.test.js (#103) (Janny)
 * Use define function in loopback-connector (jannyHou)
 * add url config example (biniam)
 * Update connector to 4.0.0 (ssh24)
 * Add doc for fitler and order (jannyHou)
 * Add advisory note regarding update (ssh24)
 * Add $elemMatch for array (jannyHou)
 * Revert "Build selector with array type data" (jannyHou)
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.

3 participants