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).
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 1405439-gitdiff.patch | 3.8 KB | jhodgdon |
| #3 | 1405438-remove-cruft.patch | 3.52 KB | jhodgdon |
Comments
Comment #1
jhodgdonI agree with the idea of deleting updates and redirecting to manage.
Comment #2
jhodgdonI 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.
Comment #3
jhodgdonHere'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.
Comment #4
jhodgdon#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.
Comment #5
jhodgdonHere 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.
Comment #6
jhodgdonAfter 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.
Comment #7
dww(edit: deleted comment I accidentally posted to the wrong issue -- sorry! -Derek)
Comment #8
dwwSorry, crap, restoring this. I'm not *actually* sure it's RTBC since I didn't look, but I'm just cleaning up after myself. ;)
Comment #9
drummCommitted, 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.
Comment #10
jhodgdonI have no problem with that change -- I was just trying to preserve the existing functionality in my patch. Thanks!
Comment #11
drummDeployed, I think earlier this week.