Problem/Motivation

After #2509566: Convert sms_send() into a service the sms_provider service was created and SMS sending functionality transferred to the class. An improvement would be to make that service swappable so that other modules can implement improved versions of the SmsProviderInterface.

Proposed resolution

Create a new sms_provider service that will be used in sms_send() and will be part of the API. sms_provider is an alias for sms_provider.default

Remaining tasks

Patch
Reviews
Commit

User interface changes

None

API changes

API addition: sms_provider service aliasing sms_provider.default which can be overridden in the services.yml file with another SmsProviderInterface

Data model changes

None

CommentFileSizeAuthor
#2 make_sms_provider-2606280-2.patch1.97 KBalmaudoh

Comments

almaudoh created an issue. See original summary.

almaudoh’s picture

almaudoh commented
Status: Active » Needs review
StatusFileSize
new make_sms_provider-2606280-2.patch1.97 KB

  • almaudoh committed 8394662 on 8.x-1.x 
    Issue #2606280: Make SMS Provider easily swappable
almaudoh’s picture

almaudoh commented
Status: Needs review » Fixed

Committed / pushed to 8.x-1.x

Status: Fixed » Closed (fixed)

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