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

Added the possibility to search for reference using OR search #451

Closed
wants to merge 2 commits into from
Closed

Added the possibility to search for reference using OR search #451

wants to merge 2 commits into from

Conversation

botunge
Copy link
Contributor

@botunge botunge commented Sep 17, 2016 •
edited

I've opened up, so that the ReferenceClientParam, can take a list of ids, used in an OR search and added a couple of tests displaying that it works in DSTU2 and STU3

@coveralls
Copy link

Coverage Status

Coverage increased (+0.007%) to 87.995% when pulling cfc3c73 on botunge:master into 364c18e on jamesagnew:master.

@jamesagnew
Copy link
Collaborator

Thanks for the contribution! Merging it now

jamesagnew added a commit that referenced this pull request Oct 12, 2016
@jamesagnew
Credit for #451
80e303b
jamesagnew added a commit that referenced this pull request Oct 12, 2016
@jamesagnew
Rename method from #451
8c455d4
@jamesagnew
Copy link
Collaborator

Ok, this has been merged. Thanks for the contribution!

FYI I have renamed the method from ids(..) to hasAnyOfIds just to keep consistent with the other methods in that class, but otherwise it's been added as is.

@jamesagnew jamesagnew closed this Oct 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@botunge @coveralls @jamesagnew