Skip to content

Fix journal routes for names containing slashes - #339

Open
vangberg wants to merge 1 commit into
mainfrom
fix-journal-slash-route
Open

vangberg wants to merge 1 commit into
mainfrom
fix-journal-slash-route

Conversation

@vangberg

@vangberg vangberg commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Allow journal names containing slashes in the journal route constraint. Add a regression test using Annales de l'Institut Pasteur / Microbiologie to verify the generated URL resolves to the full journal name.

This was surfaced by exceptions in production such as the following:

ActionView::Template::Error
No route matches {:action=>"show", :controller=>"journals", :journal=>"Annales de l'Institut Pasteur / Microbiologie"}, possible unmatched constraints: [:journal]

@vangberg vangberg self-assigned this Sep 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant