Comment 35 for bug 1749475

Revision history for this message
Michele Morgan (mmorgan) wrote :

I've done some additional testing on the latest branch of this development on the Feedback Fest test server.

My testing showed that Terran's concerns #1 and #2 in comment 25 are both resolved. #3 could be resolved by changing "Definition" to "Event Definition", but as it is, the values that populate the dropdown are easily recognized as trigger event definitions.

I would opt for a minor tweak in the stock email output trigger to remove extra white space, but as it is, the patron experience is 1000x better.

This code also fixes bug 1854986.

I will add a sign off branch soon.

The problem with adding to Trigger Event Definition Group Members stated in comment #26 is still an issue, but I would defer that issue to a separate bug. The stock groups and triggers provide plenty to work with as they are.