[Bps-public-commit] rt-extension-notifysms branch, master, updated. 3d18b752f34a9eda997fa5e0907cabab136e9e35

[Craig Kaiser]

The branch, master has been updated
       via  3d18b752f34a9eda997fa5e0907cabab136e9e35 (commit)
      from  27579624863fb938826234d4fe50357a9a9d0a05 (commit)

Summary of changes:
 Changes                       |  2 +-
 README                        | 20 +++++++++-----------
 lib/RT/Action/NotifyTwilio.pm |  7 ++-----
 lib/RT/Extension/NotifySMS.pm | 20 +++++++++-----------
 4 files changed, 21 insertions(+), 28 deletions(-)

- Log -----------------------------------------------------------------
commit 3d18b752f34a9eda997fa5e0907cabab136e9e35
Author: Craig Kaiser <craig at bestpractical.com>
Date:   Mon Jun 25 09:48:49 2018 -0400

    Update documentation for extension

diff --git a/Changes b/Changes
index c6043f5..949af25 100644
--- a/Changes
+++ b/Changes
@@ -1,4 +1,4 @@
-Revision history for RT-Extension-Text-Messages
+Revision history for RT-Extension-NotifySMS
 
 0.01 [Release Date]
  - Initial version
diff --git a/README b/README
index db2f42a..848ff94 100644
--- a/README
+++ b/README
@@ -1,21 +1,21 @@
 NAME
-    RT-Extension-Text-Messages - Provide addional actions to send text
+    RT-Extension-NotifySMS - Provide additional actions to send text
     messages.
 
 DESCRIPTION
-    Provide addional actions for Scrips, that allow for the sending of
+    Provide additional actions for Scrips, that allow for the sending of
     messages inplace of email. The new action can be loaded in the web UI
     from:
 
-    Global -> Actions -> Create
+        Global -> Actions -> Create
 
     where the action module is one of the notify option from this extension:
 
-    "NotifyTwilio"
+        "NotifyTwilio"
 
     For the action you can choose to pass the following parameters:
 
-    All, Owner, Requestor, AdminCc, Cc
+        All, Owner, Requestor, AdminCc, Cc
 
 RT VERSION
     Works with RT 4.4.0
@@ -31,10 +31,8 @@ INSTALLATION
 
             Plugin('RT::Extension::NotifySMS');
 
-        You will need to set the following config values as well as other
-        specific to the notify action being implemented:
-
-            Set(@MessageRoles, 'AdminCc');
+        You will also need to set other config values depending on which sms
+        action is being used, see the module for specifications.
 
     Clear your mason cache
             rm -rf /opt/rt4/var/mason_data/obj
@@ -47,11 +45,11 @@ AUTHOR
 BUGS
     All bugs should be reported via email to
 
-        L<bug-RT-Extension-Text-Messages at rt.cpan.org|mailto:bug-RT-Extension-Text-Messages at rt.cpan.org>
+        L<bug-RT-Extension-NotifySMS at rt.cpan.org|mailto:bug-RT-Extension-NotifySMS at rt.cpan.org>
 
     or via the web at
 
-        L<rt.cpan.org|http://rt.cpan.org/Public/Dist/Display.html?Name=RT-Extension-Text-Messages>.
+        L<rt.cpan.org|http://rt.cpan.org/Public/Dist/Display.html?Name=RT-Extension-NotifySMS>.
 
 LICENSE AND COPYRIGHT
     This software is Copyright (c) 2018 by Bestpractical Solutions
diff --git a/lib/RT/Action/NotifyTwilio.pm b/lib/RT/Action/NotifyTwilio.pm
index 738b6a0..7f62978 100644
--- a/lib/RT/Action/NotifyTwilio.pm
+++ b/lib/RT/Action/NotifyTwilio.pm
@@ -8,11 +8,8 @@ use LWP::UserAgent;
 
 =head2 NotifyTwilio
 
-Send a message using the Twilio web api, requires:
-
-    { @Recipients, $Msg }
-
-    Then uses the Twilio api config values from RT_SiteConfig.pm:
+Send a message using the Twilio web api, requires the follow
+config values from RT_SiteConfig.pm:
 
         Set($TwilioAccounId, 'Secret');
         Set($TwilioAuthToken, 'Secret');
diff --git a/lib/RT/Extension/NotifySMS.pm b/lib/RT/Extension/NotifySMS.pm
index e3f9926..69fc50b 100644
--- a/lib/RT/Extension/NotifySMS.pm
+++ b/lib/RT/Extension/NotifySMS.pm
@@ -7,22 +7,22 @@ our $VERSION = '0.01';
 
 =head1 NAME
 
-RT-Extension-Text-Messages - Provide addional actions to send text messages.
+RT-Extension-NotifySMS - Provide additional actions to send text messages.
 
 =head1 DESCRIPTION
 
-Provide addional actions for Scrips, that allow for the sending of messages
+Provide additional actions for Scrips, that allow for the sending of messages
 inplace of email. The new action can be loaded in the web UI from:
 
-Global -> Actions -> Create
+    Global -> Actions -> Create
 
 where the action module is one of the notify option from this extension:
 
-"NotifyTwilio"
+    "NotifyTwilio"
 
 For the action you can choose to pass the following parameters:
 
-All, Owner, Requestor, AdminCc, Cc
+    All, Owner, Requestor, AdminCc, Cc
 
 =head1 RT VERSION
 
@@ -46,10 +46,8 @@ If you are using RT 4.4 or greater, add this line:
 
     Plugin('RT::Extension::NotifySMS');
 
-You will need to set the following config values as well as other specific
-to the notify action being implemented:
-
-    Set(@MessageRoles, 'AdminCc');
+You will also need to set other config values depending on which
+sms action is being used, see the module for specifications.
 
 =item Clear your mason cache
 
@@ -67,11 +65,11 @@ Best Practical Solutions, LLC E<lt>modules at bestpractical.comE<gt>
 
 All bugs should be reported via email to
 
-    L<bug-RT-Extension-Text-Messages at rt.cpan.org|mailto:bug-RT-Extension-Text-Messages at rt.cpan.org>
+    L<bug-RT-Extension-NotifySMS at rt.cpan.org|mailto:bug-RT-Extension-NotifySMS at rt.cpan.org>
 
 or via the web at
 
-    L<rt.cpan.org|http://rt.cpan.org/Public/Dist/Display.html?Name=RT-Extension-Text-Messages>.
+    L<rt.cpan.org|http://rt.cpan.org/Public/Dist/Display.html?Name=RT-Extension-NotifySMS>.
 
 =head1 LICENSE AND COPYRIGHT
 

-----------------------------------------------------------------------