Everything looks fine, the last thing I'm unsure about is backporting when there are string changes. Like the commit to change the language from "Image of item" to "Book cover". Two issues with that in that backporting a string change to a release branch is generally bad form (or so I was told for i18n reasons), and also I'm not 100% sure I agree with the change in language. While many items are books, not all of them are, so leaving it general as "item" still seems most appropriate to me.
Everything looks fine, the last thing I'm unsure about is backporting when there are string changes. Like the commit to change the language from "Image of item" to "Book cover". Two issues with that in that backporting a string change to a release branch is generally bad form (or so I was told for i18n reasons), and also I'm not 100% sure I agree with the change in language. While many items are books, not all of them are, so leaving it general as "item" still seems most appropriate to me.
Thoughts or opinions before moving forward?