Skip to content

Notify admins 15 days prior to license expiration

Tyler Amos requested to merge 337250-overwrite-notify-methods-on-license into master

What does this MR do and why?

Related to #337250 (closed)

This is the second attempt at resolving the linked issue. My first attempt, !77608 (merged), had to be reverted because it was breaking reliable QA tests which wasn't seen until after the merge (see this thread for more info). Errors were raised for instances where the current license had no expiration date.

This MR adds back the logic which was reverted in the first commit. It also addresses the error caught by the reliable QA tests in the second commit.

Screenshots or screen recordings

Please see the first MR for this info.

How to set up and validate locally

Please see the first MR for this info.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Tyler Amos

Merge request reports