[8606] newsletter: remove apostrophes in subject #5775
Conversation
|
@m4ra Thanks! I've wondered why we suddenly get this error (as we didn't really change anything) and turns out it's a bug in Django 3.2.11 (because of changes in python 3.12, which we recently upgraded to with Ubuntu 24.04), updating to Django 3.2.12 fixes the error. However, using an apostrophe is as you mention still escaped as html which is wrong (but not a new bug). Should we make a new issue for that and get on with the release or fix it, this is a problem for all emails which have user entered subject, not just the newsletter here, so I guess we need to find all emails which have that in a+ and mb. |
|
@goapunk I leave this PR open then for including other instances of email subjects and remove apostrophes, besides the newsletter. And I remove it from the milestone as it will be fixed by the django update (I guess you mean 4.2.xx) @CarolingerSeilchenspringer what do you think? |
Describe your changes
Apostrophes in email subject return Unicode Error, they are now removed. Escaping them, didn't work well, as the subject gets sent with weird encoding.
fixes #5767
Tasks