Navigation Menu

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

Remove limit when choosing fields for quick search #13056

Merged
merged 1 commit into from Nov 2, 2018

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Fixes an issue with #13039 identified during review where custom fields were being inappropriately restricted

Before

Only first 25 custom fields available for quicksearch

After

All custom fields available

Technical Details

@colemanw - pre my comments on your PR

Comments

@civibot
Copy link

civibot bot commented Nov 2, 2018

(Standard links)

@civibot civibot bot added the master label Nov 2, 2018
@colemanw
Copy link
Member

colemanw commented Nov 2, 2018

Ah the old api3 limit pitfall.

@eileenmcnaughton eileenmcnaughton merged commit 1eed318 into civicrm:master Nov 2, 2018
@eileenmcnaughton eileenmcnaughton deleted the qs_follow_up branch November 2, 2018 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants