Opened 5 years ago

Closed 4 years ago

#3184 closed task (fixed)

Rewrite MailNotifier to use Data API

Reported by: dustin Owned by:
Priority: major Milestone: 0.9.0b1
Version: master Keywords:
Cc:

Description (last modified by dustin)

Blocks #2648

The MailNotifier? still exists in master, but it's still mostly expecting the old Status hierarchy to exist, rather than the new Data API.

It needs to be rewritten to listen for appropriate events from the Data API and send mail, with any additional information fetched through the Data API's "get" method.

Note that the tests for MailNotifier mock a lot of things out - incorrectly, now, so don't trust the tests!

Change History (5)

comment:1 Changed 5 years ago by dustin

  • Description modified (diff)

comment:2 Changed 5 years ago by dustin

  • Description modified (diff)

comment:3 Changed 5 years ago by dustin

  • Description modified (diff)

comment:4 Changed 5 years ago by dustin

Note (per #3227) that the generated URLs need to be considered rather carefully, too.

comment:5 Changed 4 years ago by tardyp

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.