Merge pull request #183 from intercom/jo/assign-reply-open

wip assign open close

Author: bobjflong

README.md

@@ -175,8 +175,23 @@ conversation.conversation_parts[1].body
 # REPLYING TO CONVERSATIONS
 # User (identified by email) replies with a comment
 intercom.conversations.reply(:id => conversation.id, :type => 'user', :email => 'joe@example.com', :message_type => 'comment', :body => 'foo')
-# Admin (identified by email) replies with a comment
-intercom.conversations.reply(:id => conversation.id, :type => 'admin', :email => 'bob@example.com', :message_type => 'comment', :body => 'bar')
+# Admin (identified by id) replies with a comment
+intercom.conversations.reply(:id => conversation.id, :type => 'admin', :admin_id => '123', :message_type => 'comment', :body => 'bar')
+
+# Open
+intercom.conversations.open(id: conversation.id, admin_id: '123')
+
+# Close
+intercom.conversations.close(id: conversation.id, admin_id: '123')
+
+# Assign
+intercom.conversations.assign(id: conversation.id, admin_id: '123', assignee_id: '124')
+
+# Reply and Open
+intercom.conversations.reply(:id => conversation.id, :type => 'admin', :admin_id => '123', :message_type => 'open', :body => 'bar')
+
+# Reply and Close
+intercom.conversations.reply(:id => conversation.id, :type => 'admin', :admin_id => '123', :message_type => 'close', :body => 'bar')
 
 # ASSIGNING CONVERSATIONS TO ADMINS
 intercom.conversations.reply(:id => conversation.id, :type => 'admin', :assignee_id => assignee_admin.id, :admin_id => admin.id, :message_type => 'assignment')

lib/intercom/service/conversation.rb

@@ -26,6 +26,19 @@ def reply(reply_data)
         response = @client.post("/#{collection_name}/#{id}/reply", reply_data.merge(:conversation_id => id))
         collection_class.new.from_response(response)
       end
+
+      def open(reply_data)
+        reply reply_data.merge(message_type: 'open', type: 'admin')
+      end
+
+      def close(reply_data)
+        reply reply_data.merge(message_type: 'close', type: 'admin')
+      end
+
+      def assign(reply_data)
+        assignee_id = reply_data.fetch(:assignee_id) { fail 'assignee_id is required' }
+        reply reply_data.merge(message_type: 'assignment', assignee_id: assignee_id, type: 'admin')
+      end
     end
   end
 end

spec/unit/intercom/conversation_spec.rb

@@ -18,6 +18,21 @@
     client.conversations.reply(id: '147', type: 'user', body: 'Thanks again', message_type: 'comment', user_id: 'ac4')
   end
 
+  it 'opens a conversation' do
+    client.expects(:post).with('/conversations/147/reply', { type: 'admin', message_type: 'open', conversation_id: '147', admin_id: '123'}).returns(test_conversation)
+    client.conversations.open(id: '147', admin_id: '123')
+  end
+
+  it 'closes a conversation' do
+    client.expects(:post).with('/conversations/147/reply', { type: 'admin', message_type: 'close', conversation_id: '147', admin_id: '123'}).returns(test_conversation)
+    client.conversations.close(id: '147', admin_id: '123')
+  end
+
+  it 'assigns a conversation' do
+    client.expects(:post).with('/conversations/147/reply', { type: 'admin', message_type: 'assignment', conversation_id: '147', admin_id: '123', assignee_id: '124'}).returns(test_conversation)
+    client.conversations.assign(id: '147', admin_id: '123', assignee_id: '124')
+  end
+
   # it "creates a subscription" do
   #   client.expects(:post).with("/subscriptions", {'url' => "http://example.com", 'topics' => ["user.created"]}).returns(test_subscription)
   #   subscription = client.subscriptions.create(:url => "http://example.com", :topics => ["user.created"])