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#1602 Fix join date filtering in search builder by standardis… #16578

Merged
merged 1 commit into from
Feb 18, 2020

Conversation

seamuslee001
Copy link
Contributor

…ing date field handling in Membership Query BAO

Overview

This is my alternate to #16573 and aims to do more of a standardised fix without generating deprecated notices

Before

Join date field wouldn't filter correctly within Search builder

After

Correct filter is applied when searching via Search builder using the Join date field

ping @eileenmcnaughton @lcdservices

@civibot
Copy link

civibot bot commented Feb 17, 2020

(Standard links)

@civibot civibot bot added the 5.23 label Feb 17, 2020
@lcdservices
Copy link
Contributor

@seamuslee001 I ran through a test and this is working as expected.

@eileenmcnaughton
Copy link
Contributor

good to merge on test pass then

@eileenmcnaughton
Copy link
Contributor

@seamuslee001 do you want to put up a 5.22 pr now - still trying to attract @totten's attention to a drop but this can go in I think

…ing date field handling in Membership Query BAO
@seamuslee001
Copy link
Contributor Author

test fail unrelated

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

Successfully merging this pull request may close these issues.

3 participants