This commit is contained in:
@@ -21,12 +21,14 @@
|
||||
BACKEND TODO / SYSTEM FOR RAVEN
|
||||
|
||||
|
||||
TODO: need a utility that can hash any adhoc collection of fields to work with various notify related objects
|
||||
(NOT RELIABLE UNFORTUNATELY, JUST CHECK MULTIPLE FIELDS INSTEAD)
|
||||
TODO: Never creates a duplicate APP or EMAIL notify, so checks in history log before sending
|
||||
TODO: HASH (NOT RELIABLE UNFORTUNATELY, JUST CHECK MULTIPLE FIELDS INSTEAD) uniquely identifies a notification delivery and it's logged with the history so before a new delivery is sent it's compared to the delivery log and if it was sent
|
||||
in the last 90 days then it skips it. This will avoid duplicated emails or runaway notifies etc.
|
||||
Hash needs to be unique to the destination user, teh source object and the event type exactly, if there is any difference then it should not hash the same
|
||||
TODO: check before any thing that might create a dupe
|
||||
todo: have removed the old v7 SMTP retry deliveries for simplification
|
||||
instead, it will now act as if this is always set to true
|
||||
Probe email server first if can't contact then skip smtp delivers (same as v7 when this setting true)
|
||||
New: will make every effort to err on the side of assuming delivery happened in any issue during delivery
|
||||
This will be one way to ensure it doesn't pile on the emails because of some spurious issue that actually delivered but it seemed like it might not have
|
||||
User will be able to go into notifyevent table adn remove any items they don't want delivered in case of issue
|
||||
|
||||
TODO: import v7 needs to create users for each contact and set their default for deliveries of notifications based on old region and send notification setting
|
||||
Make a case for this and or implement it with this work so it doesn't get lost
|
||||
|
||||
|
||||
Reference in New Issue
Block a user