The route for node/{node}/content-connected feels like an admin route. I propose making it to one.

Alternatives are:

  1. Outside scope of this module, add own route subscriber
  2. Inside this module settings page, add option to let site builder choose

But I generally agree the default should be an admin route given this is a tool to help site builders first and foremost.

CommentFileSizeAuthor
admin_route.patch460 bytespontus_nilsson

Comments

pontus_nilsson created an issue. See original summary.

scott_euser’s picture

Issue summary: View changes
Status: Active » Reviewed & tested by the community

This makes sense as an admin route and patch does as expected.

bletch’s picture

Version: 8.x-1.x-dev » 2.1.1
Status: Reviewed & tested by the community » Fixed
bletch’s picture

Status: Fixed » Closed (fixed)