Permitir usuario específico diferente de guardian.user en servicios de chat como Chat::CreateMessage

Lhc_fl · 12 Noviembre, 2023 12:16

Ahora que PostCreator puede especificar user como creador de la publicación y guardian para verificar los permisos de publicación, me gustaría que Discourse Chat tuviera un comportamiento similar, lo que permitiría a los plugins ayudar a enviar mensajes de un usuario (ya sea que esté o no en el canal de chat) con el guardián del usuario del sistema.

github.com/discourse/discourse

plugins/chat/app/services/chat/create_message.rb

main


      
            channel.membership_for(guardian.user)
          end
          
          def fetch_reply(params:)
            Chat::Message.find_by(id: params.in_reply_to_id)
          end
          
          def ensure_reply_consistency(channel:, params:, reply:)
            return true if params.in_reply_to_id.blank?
            reply&.chat_channel == channel
          end
          
          def fetch_thread(params:, reply:, channel:, options:)
            return Chat::Thread.find_by(id: params.thread_id) if params.thread_id.present?
            return unless reply
            reply.thread ||
              reply.build_thread(
                original_message: reply,
                original_message_user: reply.user,
                channel: channel,
                force: options.force_thread,

github.com/discourse/discourse

lib/post_creator.rb

cdbe0f74e


      
          
          def spam?
            @spam
          end
          
          def skip_validations?
            @opts[:skip_validations]
          end
          
          def guardian
            @guardian ||= @opts[:guardian] || Guardian.new(@user)
          end
          
          def valid?
            @topic = nil
            @post = nil
          
            if @user.suspended? && !skip_validations?
              errors.add(:base, I18n.t(:user_is_suspended))
              return false
            end

Uso: En Discourse Automation y Chat bridge, necesitas enviar mensajes de chat desde un plugin. Pero actualmente, es muy probable que este script falle al enviarse porque el usuario que especificas (por ejemplo, DiscoBot) puede que no esté en el canal.

github.com/discourse/discourse

plugins/chat/plugin.rb

cdbe0f74e


      
          
              script do |context, fields, automation|
                sender = User.find_by(username: fields.dig("sender", "value")) || Discourse.system_user
                channel = Chat::Channel.find_by(id: fields.dig("chat_channel_id", "value"))
          
                placeholders = { channel_name: channel.title(sender) }.merge(context["placeholders"] || {})
          
                creator =
                  ::Chat::CreateMessage.call(
                    chat_channel_id: channel.id,
                    guardian: sender.guardian,
                    message: utils.apply_placeholders(fields.dig("message", "value"), placeholders),
                  )
          
                if creator.failure?
                  Rails.logger.warn "[discourse-automation] Chat message failed to send:\n#{creator.inspect_steps.inspect}\n#{creator.inspect_steps.error}"
                end
              end
            end
          end

Sé que el equipo está muy ocupado, así que puedo hacerlo yo mismo. Espero que el equipo de Discourse considere que esto se puede agregar al núcleo, entonces me tomaré el tiempo para modificar el código aquí y enviar una solicitud de extracción.

merefield · 13 Noviembre, 2023 14:52

Discourse Chatbot resolvió un desafío similar con el código actual sin necesidad de cambios específicos en el núcleo, ¿has echado un vistazo a mi solución?

github.com/merefield/discourse-chatbot

lib/discourse_chatbot/reply_creator.rb

f782223d5


      
          # frozen_string_literal: true
          module ::DiscourseChatbot
            class ReplyCreator
          
              def initialize(options = {})
                @author = ::User.find_by(id: options[:bot_user_id])
                @guardian = Guardian.new(@author)
                @reply_to = options[:reply_to_message_or_post_id]
                @reply_to_post_number = options[:original_post_number]
                @topic_or_channel_id = options[:topic_or_channel_id]
                @message_body = options[:reply]
                @is_private_msg = options[:is_private_msg]
                @inner_thoughts = options[:inner_thoughts]
                if @message_body.blank?
                  @message_body = '...'
                end
              end

github.com/merefield/discourse-chatbot

lib/discourse_chatbot/message/message_reply_creator.rb

f782223d5


      
          module ::DiscourseChatbot
            class MessageReplyCreator < ReplyCreator
          
              def initialize(options = {})
                super(options)
              end
          
              def create
                ::DiscourseChatbot.progress_debug_message("5. Creating a new Chat Nessage...")
                begin
                  Chat::CreateMessage.call(
                    chat_channel_id: @topic_or_channel_id,
                    guardian: @guardian,
                    message: @message_body,
                  )
          
                  presence = PresenceChannel.new("/chat-reply/#{@topic_or_channel_id}")
                  presence.leave(user_id: @author.id, client_id: "12345")
          
                  ::DiscourseChatbot.progress_debug_message("6. The Message has been created successfully")
                rescue => e

Lhc_fl · 14 Noviembre, 2023 02:36

¡Muchas gracias por tu sugerencia! Déjame verla.

j.jaffeux · 4 Diciembre, 2023 19:53

Sí, como dijo @merefield, un servicio de chat acepta un tutor, y puedes configurar el tutor como quieras.

j.jaffeux · 8 Diciembre, 2023 19:54

Este tema se cerró automáticamente 4 días después de la última respuesta. Ya no se permiten nuevas respuestas.

Tema		Respuestas	Vistas
How to make only a group of users can send messages to staff Support	5	907	29 Junio 2020
Discourse Chat Plugin official , chat , included-in-core	27	30900	20 Noviembre 2025
Chat message failed to send because of channel membership Bug automation , chat	0	266	26 Octubre 2023
Introducing Discourse Chat (BETA) Announcements chat , new-feature	149	25997	3 Agosto 2022
Can't create personal chats as an admin Bug chat , fixed	2	225	25 Junio 2024

Permitir usuario específico diferente de guardian.user en servicios de chat como Chat::CreateMessage

Temas relacionados