LibriVox / librivox-catalog

LibriVox catalog and reader workflow application
https://librivox.org
MIT License
36 stars 17 forks source link

Use spacing helper in Archive uploads, and fix display of older items in the catalog #202

Closed redrun45 closed 4 months ago

redrun45 commented 4 months ago

Got admin consensus for this one: :+1:

The regex version will collapse any br that is immediately followed by a \n, a \r, or both, into just a single newline. That fixes display of old cast lists and such on our catalog pages. Added a few more test cases to address this, and possible capitalization differences.

Then we use the helper on Archive uploads, so both are consistent. Also add an extra space before and after the item description, as these are manually added by MCs right now.

redrun45 commented 4 months ago

Oh, right: per discussion in #188, I've tested this pretty thoroughly on my dev machine, including by archiving some dummy projects. I think this one is safe to go live, if you see no issues with it.