Skip to content

Conversation

daveoconnor
Copy link
Collaborator

@daveoconnor daveoconnor commented Sep 26, 2025

This PR relates to issue #1939.

Adds support for /lib/{LIBRARY} redirects to the matching library detail page.

Testing:
/lib/unordered/ should redirect to the detail page for the library set in the slug. The version it gets redirected to is based on the last visited, or latest if not previously visited.

Note that locally /lib/unordered (missing trailing slash) won't work but should work on deployed environments.

Copy link
Collaborator

@GregKaleka GregKaleka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@daveoconnor daveoconnor force-pushed the doc/1939-lib-slug-redirect branch from 45b9418 to c5da53a Compare September 30, 2025 15:55
@karimarie67
Copy link
Collaborator

tested in localhost. URL tested is
Screenshot 2025-09-30 at 16 18 30
URL directs to
Screenshot 2025-09-30 at 16 18 49

Looks good, ready to merge @daveoconnor

@daveoconnor daveoconnor force-pushed the doc/1939-lib-slug-redirect branch from c5da53a to f0bbd8b Compare September 30, 2025 21:23
@daveoconnor
Copy link
Collaborator Author

Thanks @karimarie67

Holding off on merging this for now until Rob gives the go ahead, he asked us to wait on this one.

@daveoconnor daveoconnor force-pushed the doc/1939-lib-slug-redirect branch from f0bbd8b to ad845ac Compare September 30, 2025 21:29
@karimarie67 karimarie67 added the On Hold PR is tested/ready but can't be merged yet label Sep 30, 2025
@daveoconnor daveoconnor force-pushed the doc/1939-lib-slug-redirect branch from ad845ac to 0d4dcf3 Compare October 2, 2025 00:02
@daveoconnor daveoconnor force-pushed the doc/1939-lib-slug-redirect branch from 0d4dcf3 to d7c4851 Compare October 2, 2025 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
On Hold PR is tested/ready but can't be merged yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants