CommentFileSizeAuthor
#7 simplenews.menu-links.3053098-interdiff-4-7.txt1021 bytesadamps
#7 simplenews.menu-links.3053098-7.patch2.59 KBadamps
#4 simplenews.menu-links.3053098-interdiff-2-4.txt1.02 KBadamps
#4 simplenews.menu-links.3053098-4.patch1.54 KBadamps
#2 simplenews.menu-links.3053098-2.patch801 bytesadamps

Comments

AdamPS created an issue. See original summary.

adamps’s picture

adamps commented
Status: Active » Needs review
StatusFileSize
new simplenews.menu-links.3053098-2.patch801 bytes
adamps’s picture

adamps commented
Issue tags: +Plan to commit
adamps’s picture

adamps commented
StatusFileSize
new simplenews.menu-links.3053098-4.patch1.54 KB
new simplenews.menu-links.3053098-interdiff-2-4.txt1.02 KB

Fix some naming

Status: Needs review » Needs work

The last submitted patch, 4: simplenews.menu-links.3053098-4.patch, failed testing. View results

adamps’s picture

adamps commented
Status: Needs work » Needs review

Test fails are unrelated.

adamps’s picture

adamps commented
StatusFileSize
new simplenews.menu-links.3053098-7.patch2.59 KB
new simplenews.menu-links.3053098-interdiff-4-7.txt1021 bytes

Let's have an entity operation too

  • 66811dd committed on 8.x-1.x 
    Issue #3053098 by AdamPS: Add menu links for subscribers and newsletters
adamps’s picture

adamps commented
Status: Needs review » Fixed
Issue tags: -Plan to commit

Status: Fixed » Closed (fixed)

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