Ticket #197 (closed defect: fixed)
Too many emails sent to reviewers, when news or events content item published.
| Reported by: | andycat | Owned by: | andycat |
|---|---|---|---|
| Priority: | critical | Milestone: | 0.2.3 |
| Component: | PlumiSkin | Severity: | |
| Keywords: | Cc: | ||
| Who will test this: |
Description (last modified by andycat) (diff)
With the new code in 0.2.2-final for putting news and events into 'pending' state automatically, we have 4 emails per each new content item being published. This is wrong, and confusing, since the url inside the email is pointing to temporary places for the event.
the code in question: http://dev.plone.org/collective/changeset/74055
Attachments
Change History
comment:2 Changed 4 years ago by andycat
http://dev.plone.org/collective/changeset/78465
reduces the number of state transitions to one, hence one call of the script 'inform_reviewers.py' is called, and hence only one email is sent out. BUT this is still called before the real end of the object's setup - ie the email contains the wrong URL, the url from before the item is properly renamed.
comment:3 Changed 4 years ago by andycat
Ready for testing! http://testing.engagemedia.org/
comment:4 Changed 4 years ago by andycat
http://dev.plone.org/collective/changeset/78563
ready for testing!!
