-
Notifications
You must be signed in to change notification settings - Fork 554
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Prepare for dated newspaper page requests #11150
Conversation
@@ -0,0 +1,26 @@ | |||
@(index: services.IndexPage, nav: PreviousAndNext)(implicit request: RequestHeader) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This view isn't used so can be removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah yes, thanks. That was left over from the earliest days of this thing's existence. Should have been binned a while back. Will do that now.
Conflicts: article/app/controllers/ArticleController.scala
|
||
} | ||
|
||
private def tagExists(publication: String, tag: String) = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
might be worth renaming to bookSectionTagExists
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair point. Will do.
👍 |
Prepare for dated newspaper page requests
This PR adds handling of newspaper book and book section requests so that we no longer have to hand these off to R2.
Tests are misbehaving locally, so I'm using the PR to trigger a server-side build and test 😎