Add description to entity_notification.info.yml and correct the link provided for configuration in README.md.

CommentFileSizeAuthor
#5 interdiff.txt390 bytesxangy
#5 update-2731793-5.patch679 bytesxangy
#2 update-2731793-2.patch697 bytesxangy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

xangy created an issue. See original summary.

xangy’s picture

xangy
he/him
Primary language Hindi
Location Pune 🇮🇳
CreditAttribution: xangy at QED42 commented
Assigned: xangy » Unassigned
Status: Active » Needs review
FileSize
update-2731793-2.patch697 bytes

For description, I used the module description provided in the entity_notification's d.o project page.

Status: Needs review » Needs work

The last submitted patch, 2: update-2731793-2.patch, failed testing.

xangy’s picture

xangy
he/him
Primary language Hindi
Location Pune 🇮🇳
CreditAttribution: xangy at QED42 commented
Status: Needs work » Needs review
xangy’s picture

xangy
he/him
Primary language Hindi
Location Pune 🇮🇳
CreditAttribution: xangy at QED42 commented
Issue summary: View changes
FileSize
update-2731793-5.patch679 bytes
interdiff.txt390 bytes

Removed version from previous patch and updated issue description after reading this: https://www.drupal.org/node/2000204.

  • dawehner committed 4a53986 on 8.x-1.x authored by xangy 
    Issue #2731793 by xangy: Update entity_notification.info.yml and README....
dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer and at Chapter Three commented
Status: Needs review » Fixed

Thanks a ton! This is valuable information indeed.

Committed and pushed it.

Status: Fixed » Closed (fixed)

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