U4-10669 - 7.8. Show 301 url redirects on info tab too

Created by Jeffrey Schoemaker 10 Nov 2017, 13:53:32 Updated by Dave Woestenborghs 12 Mar 2018, 07:58:33

I would love to see that you see which 301's there are for that specific node on the new Info-tab. Currently the only way the find them is to go to the Content Dashboard and the tab "Redirect URL Management". Most of the times this is an exploded list of changed urls.

I think it would make far more sense if you see them in context of the node. I think it's really crucial for users to see what happens if they change a node name, and could decide to delete some redirects themselves.

10 Attachments

Comments

Marc Goodson 13 Jan 2018, 13:28:29

@jeffrey.schoemaker@perplex.nl I've coincidentally added something like this on a clients site... was about to add an issue to link to and found this... there is a video https://youtu.be/2fsjKIV0z3s a blog post: http://tooorangey.co.uk/posts/301-redirection-in-umbraco-it-s-a-rum-do/) and a repo: https://github.com/marcemarc/tooorangey.RumDo if that helps people see what we're talking about!


Jeffrey Schoemaker 21 Jan 2018, 12:48:49

@marcemarc That is fantastic and just what I meant. I would love to have this on the Properties-tab, because I think this it's too far away in the content menu, but this should do the trick for now.

#coreFunctionality :)!

@hartvig; What do you think about this. Is it worth creating a PR?


Niels Hartvig 21 Jan 2018, 13:08:47

It's a great idea and a great improvement. I agree it should be on the properties tab, though and would be happy to accept a PR that placed it there.


Marc Goodson 22 Jan 2018, 18:11:24

@hartvig @jeffrey.schoemaker@perplex.nl so would you just have it as a list of redirects on the properties tab? or a 'Manage Redirect's button on the properties tab, that opens the panel as in the prototype... I'm just thinking do we want to be making a sql query to the redirects table, unless people specifically 'want' the redirect info... and do you enable redirects to be added to this table manually, as per the prototype, or just list what's there for now with the ability to remove...


Dave Woestenborghs 08 Feb 2018, 08:37:16

@hartvig @jeffrey.schoemaker@perplex.nl

It seems @marcemarc tried to solve the same client issues. But we took a different approach. But the end result is the same. Editors can view and manage redirects on an content node.

I just released our version as a package : https://our.umbraco.org/projects/backoffice-extensions/redirects-viewer/

Dave


Marc Goodson 18 Feb 2018, 22:43:53

Pull Request: https://github.com/umbraco/Umbraco-CMS/pull/2452


Marc Goodson 18 Feb 2018, 22:47:39

@hartvig @jeffrey.schoemaker@perplex.nl @dawoe Have added a PR to in the first instance to list the RedirectUrls on the Info tab of a content item, needs some further discussion and tidying up of markup, position of the box etc - I'm putting the 'ability to add new urls' in a separate PR, so discussion on that doesn't get in the way of this obvious editor quick win!


Søren Kottal 19 Feb 2018, 07:45:50

Great stuff @marcemarc

Would be awesome if that panel could be extensible so other redirect packages' redirects to the node could be shown too.


Jeffrey Schoemaker 19 Feb 2018, 07:46:46

Hi Marc, great stuff! Thanks for the awesome PR


Marc Goodson 19 Feb 2018, 09:18:47

@Søren.Kottal that would probably be awesome, what makes this so straightforward for the Redirect Url Management dashboard though is that the redirects are stored as oldUrl -> Content Key Guid - so when you are on a particular node, you obviously have that Guid, and can retrieve redirects easily... with external packages they tend to store the actual 'url' that will be redirected to, so it might be a little bit hit and miss, would need a bit of investigation, and discussion with the core team - possibly a bigger discussion about customising the info tab generally - part of me thinks that would be a step backwards - remember the 'properties' tab, when you could add 'everything' to that! - but let's try to get this simple super enhancement in first, further complication can come later! - (also probably out of my league to make this 'pluggable' and 'extensible'anyways :-))


Søren Kottal 19 Feb 2018, 12:50:29

I agree, make it work then make it better :)


Marc Goodson 11 Mar 2018, 19:34:11

@jeffrey.schoemaker@perplex.nl @hartvig This is the separate PR, that includes managing the 'redirects' via a separate dialog, https://github.com/umbraco/Umbraco-CMS/pull/2511 I kind of like this approach, as it solves the problem of having the 'remove' redirect button visible on the info panel - eg we're meant to be providing a way for editors to easily see what redirects exist, and not making the Info panel 'all about redirects' - so now we can list the redirects in a nicer less dominating way... (part of me thinks they should be in the same box as urls) - anyway, there is now a more 'subtle' manage redirects button, that when pressed opens a dialog, from which you can remove the redirects, or indeed controversially add new ones, (but only ones that could be created by renaming)...


Marc Goodson 11 Mar 2018, 21:39:23

screenshots


Dave Woestenborghs 12 Mar 2018, 07:58:33

@marcemarc @jeffrey.schoemaker@perplex.nl @Søren.Kottal would be nice if we could set rights on user groups on who can see the Manage redirects button


Priority: Normal

Type: Feature (request)

State: Submitted

Assignee:

Difficulty: Normal

Category:

Backwards Compatible: True

Fix Submitted: Pull request

Affected versions: 7.8.0

Due in version:

Sprint:

Story Points:

Cycle: