Problem/Motivation

Drupal Core has a new API for form validation using #config_target, see https://www.drupal.org/node/3373502

Proposed resolution

Update \Drupal\mailchimp_transactional\Form\AdminSettingsForm to use #config_target

Remaining tasks

  1. Write a merge request
  2. Review
  3. Commit

User interface changes

None

API changes

#config_target requires drupal/core 10.2.0 or higher

Data model changes

None

Issue fork mailchimp_transactional-3576208

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:

About issue forks

Comments

idebr created an issue. See original summary.

idebr opened merge request !35

idebr’s picture

idebr commented
Status: Active » Needs review

The merge request implements the following changes:

  1. \Drupal\mailchimp_transactional\Form\AdminSettingsForm now uses #config_target
  2. The minimum Drupal Core version is now 10.2
dieterholvoet’s picture

dieterholvoet commented
Status: Needs review » Postponed

Thanks for the MR, but dropping support for Drupal 9, 10 and 10.1 just for this change isn't really worth it IMO, so postponing for now.

idebr’s picture

idebr commented
Title: Use #config_target for AdminSettingsForm » [Drupal 10.2+] Use #config_target for AdminSettingsForm