Problem/Motivation

The 6.x and 7.x versions had a confirmation step that summarized who the mail would be sent to, etc.

Proposed resolution

Create a review/summary screen utilizing similar logic to the NodeForm's preview functionality.

(Ctools was considered but its wizard is probably overkill for the purposes here.)

Remaining tasks

User interface changes

API changes

Data model changes

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhedstrom created an issue. See original summary.

jhedstrom’s picture

Issue summary: View changes
sukanya.ramakrishnan’s picture

Submitting a patch for this functionality. This patch should be applied only after applying https://www.drupal.org/files/issues/2878952-Category-selection-optional-...

sukanya.ramakrishnan’s picture

Status: Active » Postponed

Postponing this issue since the patch submitted depends on https://www.drupal.org/files/issues/2878952-Category-selection-optional-...

sukanya.ramakrishnan’s picture

Submitting another patch since the new files in this patch dont apply for some reason.

sukanya.ramakrishnan’s picture

Submitting updated patch with tests based on latest dev version

sukanya.ramakrishnan’s picture

Status: Postponed » Needs review

The last submitted patch, 3: 2873779-add-confirm-form-before-sending-03.patch, failed testing.

The last submitted patch, 5: 2873779-add-confirm-form-before-sending-05.patch, failed testing.

jhedstrom’s picture

Status: Needs review » Fixed
FileSize
6.11 KB

This looks good to me, thanks!

I've committed #6 with some code style fixes (see attached interdiff).

Status: Fixed » Closed (fixed)

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