Problem/Motivation

This issue is to set the module status to deprecated.

The removal of extension Contact module was approved in #3476879: [Policy] Move Contact module to contrib.

Remaining tasks

  1. Create a section on Deprecated and obsolete modules and themes to provide recommendations for sites using extension Contact. The recommendations are to include instructions for sites using the extension and for contributed projects that depend on then extension.
  2. Add the following to the extension info.yml file
  3. lifecycle: deprecated
    lifecycle_link: https://www.drupal.org/node/3223395#s-contact

  4. Add #[IgnoreDeprecations] to all tests in the extension.
  5. Add a change record and include a link the the doc page.
  6. Deprecate any library that is only used by the extension.
  7. Do a thorough search of core for any remaining references to the extension. If references are found, outside of the extension, then creates issues to remove the references.

Release notes snippet

Issue fork drupal-3520470

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

quietone created an issue. See original summary.

quietone’s picture

Issue summary: View changes
quietone’s picture

Version: 11.1.x-dev » 11.x-dev

andypost made their first commit to this issue’s fork.

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.

andypost’s picture

Issue summary: View changes
Status: Postponed » Needs review

Created CR https://www.drupal.org/node/3571602 and rebased MR, added #[IgnoreDeprecations] to all tests (except unit one which does not install modules) and there's no libs to deprecate. Looks ready to go

godotislate’s picture

Status: Needs review » Reviewed & tested by the community

Confirmed that the relevant tests had the IgnoreDeprecations attribute added, except the Unit tests as noted and the existing other tests already deprecated with migrate_drupal.

I saw that there is still contact module reference/use in the <olivero_test module and core/tests/Drupal/Nightwatch/Tests/Olivero/oliveroPreprocessFieldMultipleValueFormTest.js test, so I commented in the parent meta issue #3520460-46: [meta] Tasks to deprecate the Contact module.

In the CR:

If you want to keep using the functionality provided by History, read the recommendations for Contact.

I assumed that "History" should be "Contact", so I made the change.

Otherwise, lgtm.

  • longwave committed 0fe642ba on 11.x
    task: #3520470 Deprecate the Contact module
    
    By: quietone
    By: andypost...

  • longwave committed 296ad93b on main
    task: #3520470 Deprecate the Contact module
    
    By: quietone
    By: andypost...
longwave’s picture

Version: main » 11.x-dev
Status: Reviewed & tested by the community » Fixed

I found no other references, it's time to go.

Committed and pushed 296ad93bb0c to main and 0fe642ba1bd to 11.x. Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

andypost’s picture

Thank you a lot! going to work on deprecation!

andypost’s picture

Thank you a lot! going to work on removal!

quietone’s picture

Issue tags: +11.4.0 release notes

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.