Implement new PG HTML <title> field format#15
Implement new PG HTML <title> field format#15cpeel merged 2 commits intoDistributedProofreaders:masterfrom rtonsing:master
Conversation
windymilla
left a comment
windymilla
left a comment
There was a problem hiding this comment.
I think it should do a slightly stronger check as the current code would allow through the old format.
Maybe at line 750, something like
if not t3.endswith(" | Project Gutenberg"):
You also wouldn't need the check below for the title ending with a full stop.
|
@cpeel Making this live at ppwb should probably be done after (not before) forum/news announcements of the change by Linda or people will get the warning and not know why |
|
Could we push this to the test site to try out? And the previous version to PROD? |
|
TEST and PROD are now running the latest ppwb, ppcomp, pphtml, and pptext. After this gets merged in I'm happy to push it out to TEST. |
|
TEST is now running the code from this PR. |
|
Behavior appears correct on TEST |
3 participants
To test, remove title or leave out "Gutenberg".