Tasks:

CommentFileSizeAuthor
#6 remove-sms-track-2707643.patch68.76 KBdpi

Comments

dpi created an issue. See original summary.

dpi’s picture

dpi
Location Perth, Australia
commented
Issue summary: View changes

After looking through sms_track, I dont think there are any reasons to keep it? So long as aformentioned tasks are completed.

dpi’s picture

dpi
Location Perth, Australia
commented
Title: [meta] Remove deprecated SMS Track functions » [meta] Remove SMS Track

Changed title to reflect my new intentions based on reviewing existing code.

dpi’s picture

dpi
Location Perth, Australia
commented

Removing SMS Track means that we will not have to bug fix the only failing Postgres tests in SMS Track: #2688247: Patch testing issue

dpi’s picture

dpi
Location Perth, Australia
commented
Issue tags: +beta blocker

API + schema removal should be addressed before beta.

dpi’s picture

dpi
Location Perth, Australia
commented
Status: Active » Needs review
StatusFileSize
new remove-sms-track-2707643.patch68.76 KB

https://github.com/dpi/smsframework/pull/34

rm -rf modules/sms_track

  • dpi committed 82ea7a1 on 8.x-1.x 
    Issue #2707643 by dpi: [meta] Remove SMS Track
dpi’s picture

dpi
Location Perth, Australia
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.