Add support for required adjusters

Created on 12 August 2022, almost 3 years ago
Updated 13 June 2025, about 1 month ago

Problem/Motivation

The Mailer attribute can set common_adjusters which are given priority in the GUI. This can be a useful hint, but actually we are much more interested in knowing which settings are required. We can then generate an error message if they are missing. Examples:

  • 'To' on contact form
  • 'Body' and 'Subject' on user emails

Proposed resolution

  • Change common_adjusters to required_config. Remove any specific values from the Mailer classes in this module that are recommended but not required (there aren't many cases).
  • Change the values from "email_XXX" to just "XXX", e.g. "subject" or "body".
  • In the policy edit form, pre-populate all the adjusters corresponding to required config and don't allow them to be removed.

Remaining tasks

User interface changes

API changes

Data model changes

Feature request
Status

Active

Version

2.0

Component

Code

Created by

🇬🇧United Kingdom adamps

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024