Skip to content
This repository has been archived by the owner on Mar 11, 2022. It is now read-only.

71 support selector #80

Merged
merged 1 commit into from
Sep 12, 2018
Merged

71 support selector #80

merged 1 commit into from
Sep 12, 2018

Conversation

smithsz
Copy link

@smithsz smithsz commented Sep 11, 2018

Checklist

  • Tick to sign-off your agreement to the Developer Certificate of Origin (DCO) 1.1
  • Added tests for code changes or test/build only changes
  • Updated the change log file (CHANGES.md|CHANGELOG.md) or test/build only changes
  • Completed the PR template below:

Description

Support the use of selectors when filtering the changes feed.

Fixes #71.

Approach

Ensure that the database confirm request is a GET.

Schema & API Changes

No change.

Security and Privacy

No change.

Testing

Includes additional test. See test/issues/cloudant-follow/71.js.

Monitoring and Logging

No change.

Copy link
Member

@ricellis ricellis left a comment

Choose a reason for hiding this comment

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

+1

@emlaver emlaver self-requested a review September 11, 2018 13:49
@smithsz smithsz merged commit 5e2e777 into master Sep 12, 2018
@smithsz smithsz deleted the 71-support-selector branch September 12, 2018 09:22
@blakew
Copy link

blakew commented Sep 26, 2018

woohoo! Thanks to you guys

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

Successfully merging this pull request may close these issues.

Support for _selector filter
4 participants