mirror of
https://github.com/abhinavxd/libredesk.git
synced 2025-11-03 05:23:48 +00:00
feat: Enable agents to create conversations from the UI
Before this feature the only way to create a conversation was by adding inbox and sending an email. Agents first search contacts by email, see a dropdown select an existing contact or fill a new email for new contact. The backend creates contact if it does not exist, creates a conversation, sends a reply to the conversation. Optinally assigns conversation to a user / team. fix: Replies to emails create a new conversation instead of attaching to the previous one. Was not happening in gmail, as gmail was sending the references headers in all replies and I missed this completely. So when libredesk searches a conversation by references headers it worked! Instead the right way is to generate the outgoing email message id and saving it in DB. This commit fixes that. There could be more backup strategies like putting reference number in the subject but that can be explored later. chore: new role `conversatons:write` that enables the create conversations feature for an agent. chore: migrations for v0.4.0.
This commit is contained in:
@@ -313,11 +313,10 @@ func (m *Manager) SendPrivateNote(media []mmodels.Media, senderID int, conversat
|
||||
}
|
||||
|
||||
// SendReply inserts a reply message in a conversation.
|
||||
func (m *Manager) SendReply(media []mmodels.Media, senderID int, conversationUUID, content string, cc, bcc []string, meta map[string]interface{}) error {
|
||||
func (m *Manager) SendReply(media []mmodels.Media, inboxID, senderID int, conversationUUID, content string, cc, bcc []string, meta map[string]interface{}) error {
|
||||
// Save cc and bcc as JSON in meta.
|
||||
cc = stringutil.RemoveEmpty(cc)
|
||||
bcc = stringutil.RemoveEmpty(bcc)
|
||||
|
||||
// Save cc and bcc as JSON in meta.
|
||||
if len(cc) > 0 {
|
||||
meta["cc"] = cc
|
||||
}
|
||||
@@ -328,6 +327,19 @@ func (m *Manager) SendReply(media []mmodels.Media, senderID int, conversationUUI
|
||||
if err != nil {
|
||||
return envelope.NewError(envelope.GeneralError, "Error marshalling message meta", nil)
|
||||
}
|
||||
|
||||
// Generage unique source ID i.e. message-id for email.
|
||||
inbox, err := m.inboxStore.GetDBRecord(inboxID)
|
||||
if err != nil {
|
||||
return err
|
||||
}
|
||||
sourceID, err := stringutil.GenerateEmailMessageID(conversationUUID, inbox.From)
|
||||
if err != nil {
|
||||
m.lo.Error("error generating source message id", "error", err)
|
||||
return envelope.NewError(envelope.GeneralError, "Error generating source message id", nil)
|
||||
}
|
||||
|
||||
// Insert Message.
|
||||
message := models.Message{
|
||||
ConversationUUID: conversationUUID,
|
||||
SenderID: senderID,
|
||||
@@ -339,6 +351,7 @@ func (m *Manager) SendReply(media []mmodels.Media, senderID int, conversationUUI
|
||||
Private: false,
|
||||
Media: media,
|
||||
Meta: string(metaJSON),
|
||||
SourceID: null.StringFrom(sourceID),
|
||||
}
|
||||
return m.InsertMessage(&message)
|
||||
}
|
||||
@@ -391,7 +404,7 @@ func (m *Manager) RecordAssigneeUserChange(conversationUUID string, assigneeID i
|
||||
}
|
||||
|
||||
// Assignment to another user.
|
||||
assignee, err := m.userStore.Get(assigneeID)
|
||||
assignee, err := m.userStore.GetAgent(assigneeID)
|
||||
if err != nil {
|
||||
return err
|
||||
}
|
||||
@@ -675,11 +688,8 @@ func (m *Manager) findOrCreateConversation(in *models.Message, inboxID, contactC
|
||||
conversationUUID string
|
||||
)
|
||||
|
||||
// Search for existing conversation.
|
||||
sourceIDs := in.References
|
||||
if in.InReplyTo != "" {
|
||||
sourceIDs = append(sourceIDs, in.InReplyTo)
|
||||
}
|
||||
// Search for existing conversation using the in-reply-to and references.
|
||||
sourceIDs := append([]string{in.InReplyTo}, in.References...)
|
||||
conversationID, err = m.findConversationID(sourceIDs)
|
||||
if err != nil && err != errConversationNotFound {
|
||||
return new, err
|
||||
|
||||
Reference in New Issue
Block a user