OpenCATS icon indicating copy to clipboard operation
OpenCATS copied to clipboard

Email templates user signature

Open skrchnavy opened this issue 8 years ago • 2 comments

Added an e-mail signature variable into e-mail templates, which is configurable for each user and from the GUI under My Profile settings (so it is available to everyone, not just admins). It utilizes the CKEditor.

Limitation: 255 chars might be enough for plain text, but might not be for HTML formatted text - this could be extended but schema update is required.

Example of email signature:

My Name | 0900 200 300 | myWebsite.com
[email protected]

skrchnavy avatar Apr 27 '17 12:04 skrchnavy

this is now targeting a php7 feature branch. It was originally on the php5 master branch. Do I need tor rebase & merge if/when accepting it?

RussH avatar Oct 01 '19 16:10 RussH

My observations: 1- Indeed, it's been couple of years held behind. It certainly needs a rebase. 2- I concur with Codacy's red flagged issues. However, the project is infected with bad coding habits and free will usage of the global variables. 3- I also concur with Codacy's unnecessary usage of "else" statement.

My recommendation: 1- Since, there has been none enforcing the best coding practice, I'm tempted to push for the implementation of singleton getters in CATSUtility.php, once and for good, to resolve the references to the required global variables. This project needs to get in shape.

shantadam avatar Oct 15 '19 00:10 shantadam