Tuesday, March 21, 2023
2 changes · master
Resolved issues and error corrections
This fixes an issue where a tiny tracking image in mass mailing emails could trigger a server error after emails were sent. Email open tracking can now work without disrupting recipients or creating unnecessary error logs.
Original PR description
Bug === Since 6185f14807bf527652cd928bf040fd7c596f77ca we check the <mail.mail> existence before marking the <mailing.trace> as opened, but since 57ae1b9b8b61f5f4719a8a81e9d0d21fab58cfda we remove the <mail.mail> automatically when we send them. The result is that this endpoint always raise a 500 error. To be: the <mail.mail> existence shouldn't be checked in this endpoint (the token is valid for the raw integer id). Task-3234519
Fixed an issue in Knowledge where choosing a random icon for an article could select emojis that should be blocked. Random article icons now use only approved emojis, helping keep content presentation appropriate for users.
Original PR description
This commit fixes a bug introduced in c33fe997f270a086150b628ab1021e9469e89c02 where the addIcon function was modified to accomodate the Discuss refactor. Before this commit when adding a random icon to an article, the random selection of icons would always return the banned emojis, which is the opposite of the desired behavior. Now the icon is always an authorized emoji. task-3239114