fix: Attach instagram images with file type story_mentions (#4100)

2022-03-10 20:27:30 +05:30
parent b3545f42f1
commit 647efa12e7
11 changed files with 219 additions and 18 deletions
--- a/app/builders/messages/messenger/message_builder.rb
+++ b/app/builders/messages/messenger/message_builder.rb
@@ -1,10 +1,14 @@
 class Messages::Messenger::MessageBuilder
+  include ::FileTypeHelper
+
  def process_attachment(attachment)
    return if attachment['type'].to_sym == :template

    attachment_obj = @message.attachments.new(attachment_params(attachment).except(:remote_file_url))
    attachment_obj.save!
    attach_file(attachment_obj, attachment_params(attachment)[:remote_file_url]) if attachment_params(attachment)[:remote_file_url]
+    fetch_story_link(attachment_obj) if attachment_obj.file_type == 'story_mention'
+    update_attachment_file_type(attachment_obj)
  end

  def attach_file(attachment, file_url)
@@ -22,7 +26,7 @@ class Messages::Messenger::MessageBuilder
    file_type = attachment['type'].to_sym
    params = { file_type: file_type, account_id: @message.account_id }

-    if [:image, :file, :audio, :video].include? file_type
+    if [:image, :file, :audio, :video, :share, :story_mention].include? file_type
      params.merge!(file_type_params(attachment))
    elsif file_type == :location
      params.merge!(location_params(attachment))
@@ -39,4 +43,31 @@ class Messages::Messenger::MessageBuilder
      remote_file_url: attachment['payload']['url']
    }
  end
+
+  def update_attachment_file_type(attachment)
+    return unless attachment.file_type == 'share' || attachment.file_type == 'story_mention'
+
+    attachment.file_type = file_type(attachment.file&.content_type)
+    attachment.save!
+  end
+
+  def fetch_story_link(attachment)
+    message = attachment.message
+    begin
+      k = Koala::Facebook::API.new(@inbox.channel.page_access_token) if @inbox.facebook?
+      result = k.get_object(message.source_id, fields: %w[story from]) || {}
+    rescue Koala::Facebook::AuthenticationError
+      @inbox.channel.authorization_error!
+      raise
+    rescue StandardError => e
+      result = {}
+      Sentry.capture_exception(e)
+    end
+    story_id = result['story']['mention']['id']
+    story_sender = result['from']['username']
+    message.content_attributes[:story_sender] = story_sender
+    message.content_attributes[:story_id] = story_id
+    message.content = I18n.t('conversations.messages.instagram_story_content', story_sender: story_sender)
+    message.save!
+  end
 end
--- a/app/helpers/file_type_helper.rb
+++ b/app/helpers/file_type_helper.rb
@@ -14,7 +14,8 @@ module FileTypeHelper
      'image/png',
      'image/gif',
      'image/bmp',
-      'image/webp'
+      'image/webp',
+      'image'
    ].include?(content_type)
  end

@@ -23,7 +24,8 @@ module FileTypeHelper
      'video/ogg',
      'video/mp4',
      'video/webm',
-      'video/quicktime'
+      'video/quicktime',
+      'video'
    ].include?(content_type)
  end
 end
--- a/app/javascript/dashboard/components/widgets/conversation/Message.vue
+++ b/app/javascript/dashboard/components/widgets/conversation/Message.vue
@@ -50,7 +50,10 @@
        <bubble-actions
          :id="data.id"
          :sender="data.sender"
+          :story-sender="storySender"
+          :story-id="storyId"
          :is-a-tweet="isATweet"
+          :has-instagram-story="hasInstagramStory"
          :is-email="isEmailContentType"
          :is-private="data.private"
          :message-type="data.message_type"
@@ -146,6 +149,10 @@ export default {
      type: Boolean,
      default: false,
    },
+    hasInstagramStory: {
+      type: Boolean,
+      default: false,
+    },
  },
  data() {
    return {
@@ -209,6 +216,12 @@ export default {
    sender() {
      return this.data.sender || {};
    },
+    storySender() {
+      return this.contentAttributes.story_sender || {};
+    },
+    storyId() {
+      return this.contentAttributes.story_id || {};
+    },
    contentType() {
      const {
        data: { content_type: contentType },
--- a/app/javascript/dashboard/components/widgets/conversation/MessagesView.vue
+++ b/app/javascript/dashboard/components/widgets/conversation/MessagesView.vue
@@ -47,6 +47,7 @@
        class="message--read"
        :data="message"
        :is-a-tweet="isATweet"
+        :has-instagram-story="hasInstagramStory"
      />
      <li v-show="getUnreadCount != 0" class="unread--toast">
        <span class="text-uppercase">
@@ -64,6 +65,7 @@
        class="message--unread"
        :data="message"
        :is-a-tweet="isATweet"
+        :has-instagram-story="hasInstagramStory"
      />
    </ul>
    <div
@@ -215,6 +217,10 @@ export default {
      return this.conversationType === 'tweet';
    },

+    hasInstagramStory() {
+      return this.conversationType === 'instagram_direct_message';
+    },
+
    selectedTweet() {
      if (this.selectedTweetId) {
        const { messages = [] } = this.getMessages;
--- a/app/javascript/dashboard/components/widgets/conversation/bubble/Actions.vue
+++ b/app/javascript/dashboard/components/widgets/conversation/bubble/Actions.vue
@@ -35,6 +35,19 @@
        size="16"
      />
    </button>
+    <a
+      v-if="hasInstagramStory && (isIncoming || isOutgoing) && linkToStory"
+      :href="linkToStory"
+      target="_blank"
+      rel="noopener noreferrer nofollow"
+    >
+      <fluent-icon
+        v-tooltip.top-start="$t('CHAT_LIST.LINK_TO_STORY')"
+        icon="open"
+        class="action--icon cursor-pointer"
+        size="16"
+      />
+    </a>
    <a
      v-if="isATweet && (isOutgoing || isIncoming) && linkToTweet"
      :href="linkToTweet"
@@ -67,6 +80,14 @@ export default {
      type: String,
      default: '',
    },
+    storySender: {
+      type: String,
+      default: '',
+    },
+    storyId: {
+      type: String,
+      default: '',
+    },
    isEmail: {
      type: Boolean,
      default: true,
@@ -79,6 +100,10 @@ export default {
      type: Boolean,
      default: true,
    },
+    hasInstagramStory: {
+      type: Boolean,
+      default: true,
+    },
    messageType: {
      type: Number,
      default: 1,
@@ -119,6 +144,13 @@ export default {
      return `https://twitter.com/${screenName ||
        this.inbox.name}/status/${sourceId}`;
    },
+    linkToStory() {
+      if (!this.storyId || !this.storySender) {
+        return '';
+      }
+      const { storySender, storyId } = this;
+      return `https://www.instagram.com/stories/${storySender}/${storyId}`;
+    },
    showSentIndicator() {
      return this.isOutgoing && this.sourceId && this.isAnEmailChannel;
    },
--- a/app/javascript/dashboard/i18n/locale/en/chatlist.json
+++ b/app/javascript/dashboard/i18n/locale/en/chatlist.json
@@ -76,6 +76,7 @@
    "RECEIVED_VIA_EMAIL": "Received via email",
    "VIEW_TWEET_IN_TWITTER": "View tweet in Twitter",
    "REPLY_TO_TWEET": "Reply to this tweet",
+    "LINK_TO_STORY": "Go to instagram story",
    "SENT": "Sent successfully",
    "NO_MESSAGES": "No Messages",
    "NO_CONTENT": "No content available",
--- a/app/models/attachment.rb
+++ b/app/models/attachment.rb
@@ -34,7 +34,7 @@ class Attachment < ApplicationRecord
  has_one_attached :file
  validate :acceptable_file

-  enum file_type: [:image, :audio, :video, :file, :location, :fallback]
+  enum file_type: [:image, :audio, :video, :file, :location, :fallback, :share, :story_mention]

  def push_event_data
    return unless file_type
--- a/app/models/message.rb
+++ b/app/models/message.rb
@@ -64,7 +64,7 @@ class Message < ApplicationRecord
  # [:deleted] : Used to denote whether the message was deleted by the agent
  # [:external_created_at] : Can specify if the message was created at a different timestamp externally
  store :content_attributes, accessors: [:submitted_email, :items, :submitted_values, :email, :in_reply_to, :deleted,
-                                         :external_created_at], coder: JSON
+                                         :external_created_at, :story_sender, :story_id], coder: JSON

  store :external_source_ids, accessors: [:slack], coder: JSON, prefix: :external_source_id