Wish List - New action trigger for SMS Preminder

Bug #1668734 reported by Terran McCanna
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

We developed an action trigger to send out SMS texts 3 days prior to an item's due date.

I'm working on getting it into the correct format to submit, and I'll make it disabled by default.

Revision history for this message
Terran McCanna (tmccanna) wrote :
tags: added: pullrequest
Changed in evergreen:
assignee: Terran McCanna (tmccanna) → nobody
Revision history for this message
Remington Steed (rjs7) wrote :

Terran, I just looked over your branch (I haven't tested it though), and I think it looks mostly right. One small problem I see is that instead of using an explicit upgrade script number, you should just use "XXXX" (both in the filename and in the "SELECT evergreen.upgrade_deps_block_check(..." line). The committer will fill in whatever the next number is when it's committed.

Also, there is a minor problem with your release notes entry. The header underline ("^^^^") needs to be the same length as the header text.

Besides those small issues, I think you have all the right pieces.

Revision history for this message
Terran McCanna (tmccanna) wrote :

Thank you, Remington! I'll work on the corrections.

Changed in evergreen:
assignee: nobody → Terran McCanna (tmccanna)
Revision history for this message
Terran McCanna (tmccanna) wrote :
Changed in evergreen:
assignee: Terran McCanna (tmccanna) → nobody
Revision history for this message
Chris Sharp (chrissharp123) wrote :

Signoff branch here:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/csharp/lp1668734_sms_preminder_action_trigger

Though we try as a rule to have people from different organizations do the signoffs on each other's patches, factoring in Remington's feedback and the fact that we have been running this code in PINES production for several months, I think we're good.

tags: added: signedoff
Galen Charlton (gmc)
Changed in evergreen:
assignee: nobody → Galen Charlton (gmc)
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed to master, along with a couple tweaks. Thanks, Terran and Chris!

Changed in evergreen:
status: New → Fix Committed
milestone: none → 3.0-alpha
assignee: Galen Charlton (gmc) → nobody
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.