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

(NFC) CRM-19323: Advanced search relationship date clause description… #9000

Merged
merged 2 commits into from
Sep 15, 2016

Conversation

ravi-sharma
Copy link
Contributor

…s are inaccurate.

@civicrm-builder
Copy link

Can one of the admins verify this patch?

@yashodha yashodha added the master label Sep 8, 2016
@totten
Copy link
Member

totten commented Sep 8, 2016

jenkins, ok to test

@seamuslee001
Copy link
Contributor

@ravi-sharma Ravi, I think the end date quill was correct in the current code, I.e, if we are searching for end dates from x date it would be on or after x date and the to is on or before.

Where as for start dates it is start date on or after x for where there is a value in the from box and the to date should be on or before x date

@ravi-sharma
Copy link
Contributor Author

@seamuslee001 Thanks for the notes, made few adjustments. Could you please check now?

@seamuslee001
Copy link
Contributor

Hi @ravi-sharma I have tested and the language looks good, this can be merged in now. @totten

@monishdeb monishdeb merged commit c949de0 into civicrm:master Sep 15, 2016
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.

6 participants