As suggested at #1392196: Redo Documentation section navigation, http://drupal.org/documentation/updates isn't too useful now that http://drupal.org/documentation/manage exists. As part of that issue, I'm already removing its section tab (the one next to Docs Home and API on all doc pages).

If http://drupal.org/documentation/updates can go, please go ahead and delete the node and add a redirect to http://drupal.org/documentation/manage at http://drupal.org/admin/build/path-redirect. Once that is done, send this issue over to the drupalorg project to delete the now-dead code that adds the dynamic table to the node content (search for the old nid 23192 to find code).

Comments

jhodgdon’s picture

Issue tags: +docs infrastructure

I agree with the idea of deleting updates and redirecting to manage.

jhodgdon’s picture

Issue tags: +valid issue

I have proposed a couple of new additions to the /manage view on #1299822: Update Docs Management view that will make it really replace the documentation/updates page. I think we should wait until that issue gets to "fixed" before we remove the updates page.

jhodgdon’s picture

StatusFileSize
new3.52 KB

Here's a potential patch that I think removes the unnecessary code. I was editing this via ssh on a dev site... not sure about special characters -- in particular these lines should have the special "..." character rather than three dots, but I have no idea how to get that into the vi editor via SSH, and all the special characters looked totally wonky there. Perhaps someone can fix the patch up. Anyway, some of that recent updates code is in use on the d.o home page, so that can't quite be removed yet.

Link to the patch:
http://docs-manage-drupal.redesign.devdrupal.org/1405438-remove-cruft.patch

I'll try to download/attach it too but the special characters could get clobbered.

jhodgdon’s picture

Project: Documentation » Drupal.org customizations
Version: » 6.x-3.x-dev
Component: Placement/Navigation/Structure » Code
Status: Active » Needs review
Issue tags: -valid issue

#1299822: Update Docs Management view has now landed.

So I went ahead and deleted node/23192, and I added redirects from the old path handbook/updates, the most recent alias documentation/updates, and node/23192 all pointing to documentation/manage.

So I am hereby submitting the patch in #3 for the drupalorg project to remove the cruft that generated this page.

jhodgdon’s picture

StatusFileSize
new3.8 KB

Here is the patch in #3 as a git diff on the latest drupalorg.

After I attach it, I'm going to test it on my new test site, and report back.

jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

After applying the patch in #5 on my new test site (http://docs-infra-drupal.redesign.devdrupal.org) and clearing the cache, I verified that:
- The home page "Docs updates" tab still works (that is in the lower section next to the big world map).
- If I'm not logged in, the "more" link at the bottom of docs updates says "More documentation" and takes me to the Community Documentation landing page (/documentation)
- If I'm logged in, the "more" link at the bottom says "More updates" and takes me to the documentation/manage view.

So I think the patch in #5 is good to go.

dww’s picture

Assigned: Unassigned » drumm
Status: Reviewed & tested by the community » Needs review

(edit: deleted comment I accidentally posted to the wrong issue -- sorry! -Derek)

dww’s picture

Assigned: drumm » Unassigned
Status: Needs review » Reviewed & tested by the community

Sorry, crap, restoring this. I'm not *actually* sure it's RTBC since I didn't look, but I'm just cleaning up after myself. ;)

drumm’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: +needs drupal.org deployment

Committed, with a small change. Subtle navigation changes for logged-in users can be confusing, so the link on the home page stays "More documentation". Maintainers can still get to documentation/manage quickly via the Documentation Team Links dashboard block or other navigation.

jhodgdon’s picture

I have no problem with that change -- I was just trying to preserve the existing functionality in my patch. Thanks!

drumm’s picture

Issue tags: -needs drupal.org deployment

Deployed, I think earlier this week.

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

  • Commit 4726f2f on 6.x-3.x, 7.x-3.x-dev by drumm:
    [#1405438] Remove Recent updates page