Follow-up to #2691343: Merge crm_core_activity_ui into crm_core_activity

Problem/Motivation

Activity type delete form class is missing.

Proposed resolution

Add delete form class for activity type and also test coverage for that.

CommentFileSizeAuthor
#3 2696733-3.patch1.96 KBsanja_m
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sanja_m created an issue. See original summary.

sanja_m’s picture

sanja_m CreditAttribution: sanja_m at MD Systems GmbH commented
Issue summary: View changes
sanja_m’s picture

sanja_m CreditAttribution: sanja_m at MD Systems GmbH commented
Status: Active » Needs review
FileSize
2696733-3.patch1.96 KB

Discussed with @slashrsm and we decided to use Drupal\Core\Entity\EntityDeleteForm for delete. Added patch with test coverage.

sanja_m’s picture

sanja_m CreditAttribution: sanja_m at MD Systems GmbH commented
Status: Needs review » Postponed

Postponed until #2691343: Merge crm_core_activity_ui into crm_core_activity is done.

sanja_m’s picture

sanja_m CreditAttribution: sanja_m at MD Systems GmbH commented
Status: Postponed » Closed (duplicate)

Test coverage is added in #2691343-28: Merge crm_core_activity_ui into crm_core_activity so closing this as duplicate.