Merge pull request #447 from OperationCode/password_change

Add slack invites

Author: apex-omontgomery

app/jobs/slack_jobs.rb

@@ -6,7 +6,7 @@ class SlackJobs < ApplicationJob
   def slack_client
     @slack_client ||= Slack::Client.new(
       subdomain: ENV.fetch('SLACK_SUBDOMAIN'),
-      token:     ENV.fetch('SLACK_TOKEN')
+      token:     ENV.fetch('SLACK_LEGACY_ADMIN_TOKEN') # admin token required to invite
     )
   end
 end

app/jobs/slack_jobs/inviter_job.rb

@@ -2,8 +2,18 @@ class SlackJobs
   class InviterJob < SlackJobs
     include Sidekiq::Worker
 
-    def perform(_email)
-      logger.debug 'Deprecated pathway, trying to determine what is placing this on queue.'
+    def perform(email)
+      # Invites user to general channel
+      Raven.user_context email: email
+      Raven.tags_context slack_invite: 'yes'
+      slack_client.invite(
+        email: email,
+        channels: ['C03GSNF77'],
+        extra_message: 'Welcome to Operation Code!\n'\
+        'Once you log in, please take a moment to introduce yourself in the #general channel - a bit about who you are, where you\'re coming from, and where you\'re interested in going next.\n'\
+        'Thank you for joining us!\n'\
+        '-from the OC volunteer team\n'\
+      )
     end
   end
 end

app/lib/slack/client.rb

@@ -20,7 +20,7 @@ def initialize(subdomain:, token:)
       @domain = "#{subdomain}.slack.com"
     end
 
-    def invite(email:, channels: [])
+    def invite(extra_message:, email:, channels: [])
       Rails.logger.info "Inviting user with email '#{email}'"
       body = send_api_request(
         to: INVITE_PATH,
@@ -29,6 +29,7 @@ def invite(email:, channels: [])
           channels:    channels.join(','),
           token:       @token,
           set_active:  'true',
+          extra_message: extra_message,
           _attempts:   1
         }
       )

app/lib/slack/utils.rb

@@ -20,7 +20,7 @@ def client
     def set_client
       Slack::Client.new(
         subdomain: OperationCode.fetch_secret_with(name: :slack_domain),
-        token:     OperationCode.fetch_secret_with(name: :slack_token)
+        token:     OperationCode.fetch_secret_with(name: :slack_legacy_admin_token)
       )
     end
   end

app/models/user.rb

@@ -151,6 +151,11 @@ def name
 
   def welcome_user
     add_to_send_grid
+    invite_to_slack
+  end
+
+  def invite_to_slack
+    SlackJobs::InviterJob.perform_async(self.email)
   end
 
   def add_to_send_grid

test/models/user_test.rb

@@ -16,6 +16,13 @@ def teardown
     assert_equal [user.id], AddUserToSendGridJob.jobs.first['args']
   end
 
+  test 'welcoming a user adds them to SlackInvite' do
+    user = create(:user, user_opts)
+    user.welcome_user
+    assert_equal 1, SlackJobs::InviterJob.jobs.length
+    assert_equal [user.email], SlackJobs::InviterJob.jobs.first['args']
+  end
+
   test 'must have a valid email' do
     refute User.new(email: 'bogusemail', password: 'password', zip: '97201').valid?
     assert User.new(email: '[email protected]', password: 'password', zip: '97201').valid?