Allow assigning self as conversation participant

Author: rsmithlal

app/controllers/better_together/conversations_controller.rb

@@ -48,8 +48,7 @@ def show # rubocop:todo Metrics/MethodLength
     private
 
     def available_participants
-      participants = Person
-                     .where.not(id: helpers.current_person.id)
+      participants = Person.all
 
       unless helpers.current_person.permitted_to?('manage_platform')
         # only allow messaging platform mangers unless you are a platform_manager

app/models/better_together/person.rb

@@ -72,6 +72,14 @@ def cover_image_variant(width, height)
       cover_image.variant(resize_to_fill: [width, height]).processed
     end
 
+    def handle
+      slug
+    end
+
+    def select_option_title
+      "#{name} - @#{handle}"
+    end
+
     def to_s
       name
     end

app/views/better_together/conversations/_conversation.html.erb

@@ -5,7 +5,7 @@
 <li id="<%= dom_id(conversation) %>" class="<%= dom_class(conversation) %> list-group-item rounded-0 <%= 'active' if conversation == @conversation %>">
   <%= link_to conversation_path(conversation), class: "stretched-link text-decoration-none d-flex justify-content-between" do %>
     <div class="mb-1 conversation-participants justify-content-between">
-      <%= render partial: 'better_together/people/mention', collection: conversation.participants.where.not(id: current_person.id), as: :person %>
+      <%= render partial: 'better_together/people/mention', collection: conversation.participants, as: :person %>
     </div>
 
     <strong><%= conversation.title if conversation.title.present? %></strong>

app/views/better_together/conversations/_conversation_content.html.erb

@@ -1,12 +1,12 @@
 <!-- app/views/conversations/_conversation_content.html.erb -->
 <div id="<%= dom_id(conversation) %>" class="conversation-messages card shadow-sm d-flex flex-column justify-content-between h-100">
-  <div class="card-header bg-secondary text-white text-center rounded-0">
-    <h4 class="mb-0">
-      <%= conversation.participants.reject { |participant| participant == current_person }.map(&:to_s).join(', ') %>
-      <% if conversation.title.present? %>
-        (<%= conversation.title %>)
-      <% end %>
-    </h4>
+  <div class="card-header bg-secondary text-white text-center d-flex align-content-center justify-content-between rounded-0">
+    <%= render partial: 'better_together/people/mention', collection: conversation.participants, as: :person %>
+    <% if conversation.title.present? %>
+      <h4 class="m-0">
+        <%= conversation.title %>
+      </h4>
+    <% end %>
   </div>
 
   <div id="conversation_messages" class="card-body p-4" style="overflow-y: auto; height: 50vh;" data-controller="better_together--conversation-messages">

app/views/better_together/conversations/new.html.erb

@@ -7,13 +7,13 @@
     <div class="card-body">
       <%= form_with(model: @conversation, local: true) do |form| %>
         <div class="mb-3">
-          <%= form.label :title %>
-          <%= form.text_field :title, class: 'form-control' %>
+          <%= form.label :participant_ids, t('.add_participants') %>
+          <%= form.collection_select :participant_ids, available_participants, :id, :select_option_title, { }, { multiple: true, class: 'form-select', required: true, data: { controller: "better_together--slim-select" } } %>
         </div>
 
         <div class="mb-3">
-          <%= form.label :participant_ids, t('.add_participants') %>
-          <%= form.collection_select :participant_ids, available_participants, :id, :name, {}, { multiple: true, class: 'form-control', required: true } %>
+          <%= form.label :title %>
+          <%= form.text_field :title, class: 'form-control' %>
         </div>
 
         <div>

app/views/better_together/people/_mention.html.erb

@@ -1,4 +1,4 @@
-<a class="text-decoration-none me-2 <%= dom_class(person, :mention) %> <%= 'me' if person == current_person %>" href="<%= person %>" id="<%= dom_id(person, :mention) %>">
+<a class="text-decoration-none me-2 <%= dom_class(person, :mention) %> <%= 'me' if person == current_person %>" href="<%= person_path(person) if person == current_person %>" id="<%= dom_id(person, :mention) %>">
   <%# Display avatar and name inline %>
   <%= profile_image_tag(person, class: 'mention', size: 25) %>
   <span class="identity-name"><%= person.name %></span>

config/locales/en.yml

@@ -11,6 +11,8 @@ en:
     contact_details:
       title: "Contact Details"
     conversations:
+      communicator:
+        create_conversation: "Create Conversation"
       conversation:
         last_message: "Last message"
       layout_with_sidebar:
@@ -20,7 +22,6 @@ en:
         new: "New"
         new_conversation: "New Conversation"
         add_participants: "Add Participants"
-        create_conversation: "Create Conversation"
       empty:
         no_messages: "No messages yet. Why not start the conversation?"
     platform_invitation:
@@ -72,7 +73,7 @@ en:
         placeholder: "Type your message..."
         send: "Send"
     new_message_notifier:
-      new_message: "New Message in %{conversation_title}"
+      new_message: "New Message"
       from_html: "From: %{sender}"
     notifications:
       index:

config/locales/es.yml

@@ -11,6 +11,8 @@ es:
     contact_details:
       title: "Detalles de Contacto"
     conversations:
+      communicator:
+        create_conversation: "Crear Conversación"
       conversation:
         last_message: "Último mensaje"
       layout_with_sidebar:
@@ -20,7 +22,6 @@ es:
         new: "Nuevo"
         new_conversation: "Nueva Conversación"
         add_participants: "Agregar Participantes"
-        create_conversation: "Crear Conversación"
       empty:
         no_messages: "Aún no hay mensajes. ¿Por qué no iniciar la conversación?"
     platform_invitation:
@@ -72,7 +73,7 @@ es:
         placeholder: "Escribe tu mensaje..."
         send: "Enviar"
     new_message_notifier:
-      new_message: "Nuevo mensaje en %{conversation_title}"
+      new_message: "Nuevo mensaje"
       from_html: "De: %{sender}"
     notifications:
       index: