On page /admin/people/simplenews there are 4 local actions: Add subscriber, Mass subscribe, Mass unsubscribe, Export. The recommendation is to have only one (according to @Berdir in #2).

Proposed resolution

Keep "Add subscriber" and move the rest to be local tasks.

The patch also adds a route for entity.simplenews_subscriber.collection which is the standard Drupal way so easier to follow the code. The entity collection doesn't have any fields so it's not very useful, but it does at least mean that if someone deletes the subscriber view then the site isn't critically broken.

CommentFileSizeAuthor
#3 simplenews.local-actions.3037266-3.patch6.09 KBadamps

Comments

AdamPS created an issue. See original summary.

berdir’s picture

Forgot there were already 3 :)

There should in fact only be one, I think there is no place in core where there is more than one by default.

PS: I'll try to comment on all the other issues soon as well.

adamps’s picture

Version: 8.x-1.x-dev » 8.x-2.x-dev
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new6.09 KB
adamps’s picture

Issue tags: +Plan to commit

  • AdamPS committed 7f2d369 on 8.x-2.x
    Issue #3037266 by AdamPS: Reduce number of local actions on subsribers...
adamps’s picture

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

Status: Fixed » Closed (fixed)

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