One of my worst WFT moments was just over 2 years ago.

A former colleague had been tasked with “upgrading” our solution for handling customer specific CSS on our platform for building newsletter emails.

He had been with us for about 5 years and ported most of the front end gui over that time from classic asp to .net and C#.

This work started in November and with a pause over dec-mid jan for high season and Christmas leave he continued.

In the beginning if mars we had the first of multiple WTF on that when I realized that his solution required a lot of special CSS or rather LESS, more than the a actual HTML for the template, and all was custom less rules that was very hard to understand.

We found that he actually never really understood how LESS worked and had tried to do things in a very backward way. Another colleague jumped in and manage to clean it up a bit so it got down to manageable levels.

Then in the end of Mars came the next bigger WTF. This is a newsletter building application. Turns out the new LESS based solution was entirely dependent on the js version of LESS and only worked when running in the browser. Guess what, the email send engine is not a browser and css classes and rules generally does not work in emails.

The new solution was impossible to integrate with the part that built and sent the emails without some very heavy rework.

Oh, and it was also completely incompatible with 12 years of old newsletters and customer templates that just did not work.

And of cause, he had not shown any of this in code reviews but rather just merged it part by part to the new version branch interleaving it with 5 months of other work.

He left the company short after.

Wtf JS.
It take two days that root of the problem is two char - "()".
Someone write isManageExp instead isManageExp()

maybe im too dumb about this fuckin js thing.

I go around all file to make sure the ouput is correct.
Damn js, damn damn