-
-
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
Migrate date field to datepicker on ChangeCaseType form #13701
Conversation
(Standard links)
|
@civicrm-builder retest this please |
@demeritcowboy does this form actually work in your client's CiviCase? I'm wondering if the breakage was recent. |
@colemanw It works in that install but note that by design it doesn't update any activities touched since creation, so only activities with is_auto=1 have the date changed. So maybe related to it not appearing to do anything in your case? |
test this please |
1 similar comment
test this please |
Tested at http://core-13701-6uxs5.test-1.civicrm.org:8001. Seems to work as expected. Tested with both changing the type and date and then also just the date and not type. |
Note I had to log in as admin since demo didn't seem to have access to anything in the cases menu or to even see the menu. That seems separate from this change though. |
Thanks for the testing @demeritcowboy |
Jenkins re test this please |
Migrates a field to datepicker and updates some old code.
Note: this form seems somewhat broken. The "reset timeline" option doesn't appear to do anything. Neither does the date field have any effect on anything as far as I can tell. But in my testing it's no more broken after this patch than it was before.