Skip to content

Small change to error notification email generation

Robert Sinton requested to merge feature/rest_exception_notification_emailing into master

This is just a very small change to reduce cognitive load for anyone reading this code.

What I did

  • Changed to use the recently-made-public notify->notify_admin() method instead of notify->set(), for code clarity.

Implications

None

Setup

None

How to test

  • Code review
  • Trigger a DF_REST_exception with a notification subject message parameter, and confirm that a notification email is still rent.

Merge request reports