Problem/Motivation

ECA is currently dependending on drupal:action, but that dependency is not required.

Steps to reproduce

Proposed resolution

Remaining tasks

Remove drupal:action from the dependency list.

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#6 3294671-3.patch272 bytesgayatri chahar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mxh created an issue. See original summary.

mxh’s picture

Issue tags: +good-first-issue, +Novice
gayatri chahar’s picture

Assigned: Unassigned » gayatri chahar
gayatri chahar’s picture

Assigned: gayatri chahar » Unassigned
gayatri chahar’s picture

gayatri chahar’s picture

Status: Active » Needs review
FileSize
272 bytes

  • jurgenhaas committed e1732f8 on 1.0.x
    Issue #3294671 by gayatri chahar: Remove drupal:action dependency
    
jurgenhaas’s picture

Status: Needs review » Fixed

Thanks a lot @gayatri chahar and congratulations on your first contribution to ECA. We've just merged this to the latest dev release.

For the next time, we'd prefer to receive contributions as a merge request in an issue fork. If you want to make yourself familiar with MRs, this link might be a good starting point: https://www.drupal.org/docs/develop/git/using-git-to-contribute-to-drupa...

Status: Fixed » Closed (fixed)

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