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.
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.
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.