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

dev/core#1230 [Dedupe] Add new api option to getduplicates, permitting a new search to be forced. #15196

Merged
merged 1 commit into from
Sep 3, 2019

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Sep 3, 2019

Overview

Extra parameter for Dedupe.getduplicates - is_force_new_search allows the api to force a refresh of cached results

Before

No way to force a refresh by api if the cache is invalidated

After

New api parameter

Technical Details

Per the inbuilt screen, sometimes there is a need to clear the cached duplicates & reload - e.g because more have been
created in the mean time.

Comments

https://lab.civicrm.org/dev/core/issues/1230

…rced.

Per the inbuilt screen, sometimes there is a need to clear the cached duplicates & reload - e.g because more have been
created in the mean time.
@civibot
Copy link

civibot bot commented Sep 3, 2019

(Standard links)

@civibot civibot bot added the master label Sep 3, 2019
@eileenmcnaughton eileenmcnaughton changed the title Add new api option to getduplicates, permitting a new search to be forced. [Dedupe] Add new api option to getduplicates, permitting a new search to be forced. Sep 3, 2019
@eileenmcnaughton eileenmcnaughton changed the title [Dedupe] Add new api option to getduplicates, permitting a new search to be forced. dev/core#1230 [Dedupe] Add new api option to getduplicates, permitting a new search to be forced. Sep 3, 2019
@mattwire
Copy link
Contributor

mattwire commented Sep 3, 2019

@eileenmcnaughton We should really have a test to lock in this parameter, but I'm aware that you're doing lot's of work in this area so I'll merge and assume coverage will come later.

@mattwire mattwire merged commit e0b4778 into civicrm:master Sep 3, 2019
@eileenmcnaughton eileenmcnaughton deleted the dedupe_api branch September 3, 2019 09:56
@eileenmcnaughton
Copy link
Contributor Author

thanks @mattwire - you are right there isn't cover on this option but there is a lot on the code around it - will see what I can do

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

Successfully merging this pull request may close these issues.

2 participants