feat: show state and responsible subject changes in issue activity

app/assets/stylesheets/application.scss

@@ -2474,6 +2474,76 @@ section.podnet .podnet-discussion .comment {
   padding: 30px 20px;
   word-break: break-word;
 }
+section.podnet .podnet-discussion .state-change-message {
+  align-items: center;
+  display: flex;
+  flex-wrap: wrap;
+  gap: 16px;
+  margin: 10px 0;
+  padding: 10px 20px;
+}
+
+section.podnet .podnet-discussion .state-change-message .change-icon {
+  align-items: center;
+  background: variables.$blue-light;
+  border-radius: 50%;
+  display: flex;
+  height: 32px;
+  justify-content: center;
+  width: 32px;
+  flex-shrink: 0;
+
+  svg {
+    stroke: variables.$blue-dark;
+    width: 16px;
+    height: 16px;
+  }
+}
+
+section.podnet .podnet-discussion .state-change-message .content {
+  align-items: center;
+  display: flex;
+  flex: 1;
+  flex-wrap: wrap;
+  gap: 10px;
+}
+
+section.podnet .podnet-discussion .state-change-message .text {
+  align-items: center;
+  color: variables.$black-light;
+  display: flex;
+  flex-wrap: wrap;
+  font-family: variables.$general;
+  font-size: variables.em(16,16);
+  gap: 8px;
+  font-weight: variables.$medium;
+}
+
+section.podnet .podnet-discussion .state-change-message .state-badge {
+  border-radius: 12px;
+  color: variables.$black;
+  font-size: variables.em(14,16);
+  font-weight: variables.$semibold;
+  padding: 4px 12px;
+}
+
+section.podnet .podnet-discussion .state-change-message .arrow-right {
+  stroke: variables.$black-light;
+  width: 16px;
+  height: 16px;
+}
+
+section.podnet .podnet-discussion .state-change-message .date {
+  color: variables.$black-light;
+  font-size: variables.em(14,16);
+  margin-left: auto;
+}
+
+section.podnet .podnet-discussion .comment-con > *:has(.state-change-message) + * .comment,
+section.podnet .podnet-discussion .comment-con > *:has(.state-change-message) + * .edit-comment-form {
+  border-top: 1px solid variables.$black;
+}
+
 section.podnet .podnet-discussion .duplicate {
   background: variables.$green-light;
 }
@@ -3402,4 +3472,4 @@ footer .footer-nav-bottom {
 @include meta.load-css("_90em.scss");
 
 // 1600px
-@include meta.load-css("_100em.scss");
+@include meta.load-css("_100em.scss");

app/jobs/notifications/publish_issue_responsible_subject_changed_job.rb

@@ -0,0 +1,15 @@
+module Notifications
+  class PublishIssueResponsibleSubjectChangedJob < ApplicationJob
+    def perform(issue, responsible_subject_id_change: [], notification_mailer: NotificationMailer)
+      return unless responsible_subject_id_change.present?
+
+      previous_rs = ResponsibleSubject.find(responsible_subject_id_change.first)
+
+      issue.subscriptions.active.each do |subscription|
+        next unless subscription.subscriber.email_notifiable?
+
+        notification_mailer.with(subscription: subscription).issue_responsible_subject_changed(previous_rs).deliver_later
+      end
+    end
+  end
+end

app/jobs/notifications/publish_issue_state_changed_job.rb

@@ -13,6 +13,10 @@ def perform(issue, state_id_change: [], notification_mailer: NotificationMailer)
           notification_mailer.with(subscription: subscription).issue_rejected.deliver_later
         when "duplicate"
           notification_mailer.with(subscription: subscription).issue_marked_as_duplicate.deliver_later
+        when "waiting"
+          notification_mailer.with(subscription: subscription).issue_waiting.deliver_later
+        when "in_progress"
+          notification_mailer.with(subscription: subscription).issue_in_progress.deliver_later
         when "resolved"
           notification_mailer.with(subscription: subscription).issue_resolved.deliver_later
         when "unresolved"

app/mailers/notification_mailer.rb

@@ -52,10 +52,24 @@ def issue_rejected
     mail
   end
 
+  def issue_in_progress
+    mail
+  end
+
   def issue_marked_as_duplicate
     mail
   end
 
+  def issue_waiting
+    mail
+  end
+
+  def issue_responsible_subject_changed(previous_rs)
+    @previous_rs = previous_rs
+    @new_rs = @issue.responsible_subject
+    mail
+  end
+
   def praise_accepted
     mail
   end

app/models/issue.rb

@@ -106,6 +106,10 @@ class Issue < ApplicationRecord
   scope :resolution_process, -> { where.not(resolution_external_id: nil) }
 
   before_save :recalculate_computed_fields
+
+  after_update :track_state_change
+  after_update :track_responsible_subject_change
+
   after_update :notify_subscribers
 
   def imported?
@@ -237,6 +241,38 @@ def reset_counters
     save
   end
 
+  def track_state_change
+    return unless saved_change_to_state_id?
+
+    previous_state_id, new_state_id = saved_change_to_state_id
+    previous_state = Issues::State.find_by(id: previous_state_id)
+    new_state = Issues::State.find_by(id: new_state_id)
+
+    return unless new_state
+
+    activity = Issues::StateChangeActivity.create!(issue: self)
+    Issues::StateChange.create!(
+      activity: activity,
+      previous_state: previous_state,
+      new_state: new_state
+    )
+  end
+
+  def track_responsible_subject_change
+    return unless saved_change_to_responsible_subject_id?
+
+    previous_rs_id, new_rs_id = saved_change_to_responsible_subject_id
+    previous_rs = ResponsibleSubject.find_by(id: previous_rs_id)
+    new_rs = ResponsibleSubject.find_by(id: new_rs_id)
+
+    activity = Issues::SystemResponsibleSubjectChangeActivity.create!(issue: self)
+    Issues::SystemResponsibleSubjectChange.create!(
+      activity: activity,
+      previous_responsible_subject: previous_rs,
+      new_responsible_subject: new_rs
+    )
+  end
+
   def notify_subscribers
     if issue_type == "praise"
       return unless saved_change_to_state_id?
@@ -251,6 +287,8 @@ def notify_subscribers
       Notifications::PublishIssueAcceptedJob.perform_later(self)
     elsif saved_change_to_state_id?
       Notifications::PublishIssueStateChangedJob.perform_later(self, state_id_change: saved_change_to_state_id)
+    elsif saved_change_to_responsible_subject_id?
+      Notifications::PublishIssueResponsibleSubjectChangedJob.perform_later(self, responsible_subject_id_change: saved_change_to_responsible_subject_id)
     end
   end
 end

app/models/issues/state_change.rb

@@ -0,0 +1,34 @@
+# == Schema Information
+#
+# Table name: issues_state_changes
+#
+#  id                 :bigint           not null, primary key
+#  hidden             :boolean          default(FALSE), not null
+#  uuid               :uuid             not null
+#  created_at         :datetime         not null
+#  updated_at         :datetime         not null
+#  activity_id        :bigint           not null
+#  new_state_id       :bigint           not null
+#  previous_state_id  :bigint
+#  triage_external_id :integer
+#
+class Issues::StateChange < ApplicationRecord
+  belongs_to :activity, class_name: "Issues::Activity", dependent: :destroy
+  belongs_to :previous_state, optional: true, class_name: "Issues::State"
+  belongs_to :new_state, class_name: "Issues::State"
+  delegate :issue, to: :activity
+
+  before_create -> { self.uuid = SecureRandom.uuid }
+
+  def author_display_name
+    "Odkaz pre starostu"
+  end
+
+  def triage_visible?
+    false
+  end
+
+  def visible?
+    !hidden
+  end
+end

app/models/issues/state_change_activity.rb

@@ -0,0 +1,15 @@
+# == Schema Information
+#
+# Table name: issues_activities
+#
+#  id             :bigint           not null, primary key
+#  dislikes_count :integer          default(0), not null
+#  likes_count    :integer          default(0), not null
+#  type           :string           not null
+#  created_at     :datetime         not null
+#  updated_at     :datetime         not null
+#  issue_id       :bigint           not null
+#
+class Issues::StateChangeActivity < Issues::Activity
+  has_one :activity_object, class_name: "Issues::StateChange", foreign_key: :activity_id, dependent: :destroy
+end

app/models/issues/system_responsible_subject_change.rb

@@ -0,0 +1,30 @@
+# == Schema Information
+#
+# Table name: issues_system_responsible_subject_changes
+#
+#  id                              :bigint           not null, primary key
+#  hidden                          :boolean          default(FALSE), not null
+#  uuid                            :uuid             not null
+#  created_at                      :datetime         not null
+#  updated_at                      :datetime         not null
+#  activity_id                     :bigint           not null
+#  new_responsible_subject_id      :bigint
+#  previous_responsible_subject_id :bigint
+#
+class Issues::SystemResponsibleSubjectChange < ApplicationRecord
+  belongs_to :activity, class_name: "Issues::Activity", dependent: :destroy
+  belongs_to :previous_responsible_subject, optional: true, class_name: "::ResponsibleSubject"
+  belongs_to :new_responsible_subject, optional: true, class_name: "::ResponsibleSubject"
+
+  delegate :issue, to: :activity
+
+  before_create -> { self.uuid = SecureRandom.uuid }
+
+  def triage_visible?
+    false
+  end
+
+  def visible?
+    !hidden
+  end
+end

app/models/issues/system_responsible_subject_change_activity.rb

@@ -0,0 +1,15 @@
+# == Schema Information
+#
+# Table name: issues_activities
+#
+#  id             :bigint           not null, primary key
+#  dislikes_count :integer          default(0), not null
+#  likes_count    :integer          default(0), not null
+#  type           :string           not null
+#  created_at     :datetime         not null
+#  updated_at     :datetime         not null
+#  issue_id       :bigint           not null
+#
+class Issues::SystemResponsibleSubjectChangeActivity < Issues::Activity
+  has_one :activity_object, class_name: "Issues::SystemResponsibleSubjectChange", foreign_key: :activity_id, dependent: :destroy
+end

app/views/issues/state_changes/_state_change.html.erb

@@ -0,0 +1,39 @@
+<% change = state_change %>
+<%= turbo_frame_tag change do %>
+  <div class="state-change-message" id="state_change_<%= change.id %>">
+
+    <div class="change-icon">
+      <svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round">
+        <path d="M8 3 4 7l4 4"/>
+        <path d="M4 7h16"/>
+        <path d="M16 21l4-4-4-4"/>
+        <path d="M20 17H4"/>
+      </svg>
+    </div>
+
+    <div class="content">
+      <div class="text">
+        <span>Stav podnetu bol zmenený z</span>
+
+        <% if change.previous_state %>
+          <span class="state-badge state-<%= change.previous_state.name.parameterize %>">
+            <%= change.previous_state.name %>
+          </span>
+        <% else %>
+          <span class="state-badge state-unknown">neznámeho stavu</span>
+        <% end %>
+
+        <svg class="arrow-right" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round">
+          <path d="M5 12h14"></path>
+          <path d="m12 5 7 7-7 7"></path>
+        </svg>
+
+        <span class="state-badge state-<%= change.new_state.name.parameterize %>">
+          <%= change.new_state.name %>
+        </span>
+      </div>
+    </div>
+
+    <div class="date"><%= l change.created_at.to_date %></div>
+  </div>
+<% end %>

app/views/issues/system_responsible_subject_changes/_system_responsible_subject_change.html.erb

@@ -0,0 +1,50 @@
+<% change = system_responsible_subject_change %>
+<%= turbo_frame_tag change do %>
+  <div class="state-change-message" id="system_rs_change_<%= change.id %>">
+
+    <div class="change-icon">
+      <svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round">
+        <rect x="4" y="2" width="16" height="20" rx="2" ry="2"></rect>
+        <path d="M9 22v-4h6v4"></path>
+        <path d="M8 6h.01"></path>
+        <path d="M16 6h.01"></path>
+        <path d="M12 6h.01"></path>
+        <path d="M12 10h.01"></path>
+        <path d="M12 14h.01"></path>
+        <path d="M16 10h.01"></path>
+        <path d="M16 14h.01"></path>
+        <path d="M8 10h.01"></path>
+        <path d="M8 14h.01"></path>
+      </svg>
+    </div>
+
+    <div class="content">
+      <div class="text">
+        <span>Zodpovedný subjekt bol zmenený z</span>
+
+        <% if change.previous_responsible_subject %>
+          <span class="state-badge state-unknown" style="background-color: #f1f5f9; color: #475569;">
+            <%= change.previous_responsible_subject.subject_name %>
+          </span>
+        <% else %>
+          <span class="state-badge state-unknown">neznámeho subjektu</span>
+        <% end %>
+
+        <svg class="arrow-right" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round">
+          <path d="M5 12h14"></path>
+          <path d="m12 5 7 7-7 7"></path>
+        </svg>
+
+        <% if change.new_responsible_subject %>
+          <span class="state-badge state-unknown" style="background-color: #e2e8f0; color: #1e293b; font-weight: 600;">
+            <%= change.new_responsible_subject.subject_name %>
+          </span>
+        <% else %>
+          <span class="state-badge state-unknown">nikoho</span>
+        <% end %>
+      </div>
+    </div>
+
+    <div class="date"><%= l change.created_at.to_date %></div>
+  </div>
+<% end %>

app/views/notification_mailer/issue_in_progress.html.erb

@@ -0,0 +1,14 @@
+<%= ops_title "Podnet je v riešení" %>
+<p class="ops-email-paragraph">
+  <% if @issue.author == @user %>
+    Dobrý deň, samospráva začala aktívne pracovať na Vašom podnete <%= ops_issue_link @issue %>.
+  <% else %>
+    Dobrý deň, samospráva začala aktívne pracovať na podnete <%= ops_issue_link @issue %>, ktorý sledujete.
+  <% end %>
+  Detailné informácie o postupe prác nájdete v <b><%= link_to "detaile podnetu", issue_url(@issue), class: "ops-email-link", target: "_blank" %></b>.
+</p>
+<p class="ops-email-paragraph">
+  <%= ops_button "Reagovať na nový status", "#{issue_url(@issue)}#new_issue_activity" %>
+</p>
+<%= render "signature" %>
+<%= render "unsubscribe" %>