- Issue created by @quietone
- πΊπΈUnited States smustgrave
Did a small review but instead of deleting some of the tests that were using contact we probably need to find a replacement so we aren't losing functionality coverage for other things that happened to be using contact
- πΊπΈUnited States smustgrave
Actually majority is out of scope too, should be about just removing from standard profile not fixing all tests
- π¦π²Armenia hhvardan
I rolled back the previous commits that included migration tests and other changes that were out of scope. The current implementation now focuses specifically on removing Contact from the Standard profile.
- πΊπΈUnited States smustgrave
Still appears to be touching migration tests
- π¦π²Armenia hhvardan
Moved Contact to missing paths in review tests and enabled Contact module in upgrade tests.
- πΊπΈUnited States smustgrave
Iβll have to take a look later not sure why all the migrations need to be updated if we are just removing a module from a profile.
The Needs Review Queue Bot β tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide β to find step-by-step guides for working with issues.
- π¦π²Armenia hhvardan
Merged latest 11.x core and resolved conflicts. Restored the feedback_contact_form recipe.