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#4112 stop installing legacycustomsearches on new installs #27400
Conversation
🤖 Thank you for contributing to CiviCRM! ❤️ We will need to test and review this PR. 👷 Introduction for new contributors...
Quick links for reviewers...
|
The issue associated with the Pull Request can be viewed at https://lab.civicrm.org/dev/core/-/issues/4112 |
Probably here too: civicrm-core/distmaker/core-ext.txt Line 27 in 8114224
|
I think that line puts it in the tarball - which we still want |
ok. Then this seems fine. |
There's an outstanding regen needed anyway from the state/province update. |
@jaapjansma FYI - but I'm pretty sure you adapted your handling for the possibility of this being uninstalled about a year or so back |
@eileenmcnaughton I don't know what you are talking about. I do remember an issue with data processor and the lagacy custom search extension. But not sure which civi version that was one. I have clients which are still on Civi Version 5.37 or even Civi 5.14 By the way I don't have clients who use the latest versions of civicrm. Most of the installs are a couple of years old. Because at my clients testing an upgrade usually takes a couple of months. So they only upgrade once in the few years. |
@jaapjansma ok that's fine - it was just the dependency management side of it that would have affected you earlier on |
Overview
dev/core#4112 stop installing legacycustomsearches on new installs
https://lab.civicrm.org/dev/core/-/issues/4112
Before
We are most of the way through separating legacy custom searches from core & we can definitely run with them disabled - so now we are ready to stop installing on new installs
After
No change on existing installs but not enabled by default on new installs
Technical Details
@seamuslee001 what more do we need to do? regen?
Comments