-
-
Notifications
You must be signed in to change notification settings - Fork 825
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
Jcalendar replacement for event date #15661
Conversation
(Standard links)
|
c08f024
to
341bc03
Compare
This seems to work at least for searching i note there is a change in qill, Tho smart groups haven't been converted right i think. I created 4 separate smart groups. The first 2 were from the standalone find participants page and the last from advanced search. The Form Values are below.
|
@seamuslee001 did you decide what the issue was with the conversion? Or you didn't dig that far |
@@ -130,6 +130,7 @@ public function upgrade_5_20_alpha1($rev) { | |||
'mailing_job_start_date', | |||
'relationship_start_date', | |||
'relationship_end_date', | |||
'event_date', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah should be just event not event date
@seamuslee001 does that fix it? |
@eileenmcnaughton it seems to me to be threefold 1) the smart groups created by action on the stand alone find participants search doesn’t match the format when done through advanced search 2) the check in is the relative date isn’t in the relative dates array is causing issues with no.4 because the relative = 0 is not being added to the form values and 3) the relative dates upgrade was tricky. |
We just need a special routine for this group? Do you have it clear in your head how it would look? |
28e8c43
to
afe8e63
Compare
@seamuslee001 I've pulled the conversion bit out of this PR. I figure we are so close now I'll look at the last relationship field & do that & then log_date one & then check the conversion as the last step since it seems it needs to be custom |
afe8e63
to
936eded
Compare
Same as civicrm#15661 Conversion excluded for now
Same as civicrm#15661 Conversion excluded for now
I tested this and confirmed that it works noting we will handle upgrade separately |
Same as civicrm#15661 Conversion excluded for now
Same as civicrm#15661 Conversion excluded for now
Same as civicrm#15661 Conversion excluded for now
Overview
Remove jcalendar for event date in advanced search & event search
Before
Jcalendar
After
Datepicker
Technical Details
@seamuslee001 alternate approach to even_end_date - the same would also be used for relationship active date. The advantage is we keep the submitted field 'intact' rather than wrangling it.
Comments
I didn't rename the field - but we could.