Specifically the devel gateway needs major work.

We should decide what stays and what goes.

more to come,...

Tasks

CommentFileSizeAuthor
#6 testsms.jpg176.32 KBdpi
#6 sms-devel-update-2644900.patch39.92 KBdpi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dpi created an issue. See original summary.

dpi’s picture

I've already done some work over at https://github.com/dpi/smsframework/pull/14/.

Edit 13 April 2016: added link to PR

almaudoh’s picture

Virtual gateway is under sms_devel.module, so added that issue as a child of this one.

dpi’s picture

Issue summary: View changes

Adding child task to issue summary

dpi’s picture

Issue tags: +beta blocker

Minimum functionality should be implemented, and corners should be rounded off before beta.

dpi’s picture

Status: Active » Needs review
FileSize
39.92 KB
176.32 KB

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

Test sms form

The send form has been updated with a couple of extra options. Test coverage updated.

The PR removes the virtual gateway, since I'm still not sure about #2642034: Port virtual gateway to Drupal 8?.

almaudoh’s picture

The PR removes the virtual gateway, since I'm still not sure about #2642034: Port virtual gateway to Drupal 8?.

That's ok. We can put it back in that issue if we find there is still a need for it.

  • dpi committed 7af8757 on 8.x-1.x
    Issue #2644900 by dpi: Update SMS Devel submodule
    
    Removed SMS virtual...
dpi’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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