feat: add back button

Author: “Cengiz

Gemfile.lock

@@ -85,7 +85,7 @@ GEM
     bindex (0.8.1)
     bootsnap (1.19.0)
       msgpack (~> 1.2)
-    brakeman (7.1.1)
+    brakeman (7.1.2)
       racc
     builder (3.3.0)
     bundler-audit (0.9.3)

app/components/confirm_back_phase_modal_component.html.erb

@@ -0,0 +1,66 @@
+<div data-controller="dialog" data-dialog-modal-value="true">
+  <%# Trigger Button %>
+  <button type="button"
+          data-action="click->dialog#open"
+          class="h-10 sm:h-12 px-4 sm:px-8 font-bold uppercase tracking-widest text-xs sm:text-sm border-[2px] rounded-[1px] transition-all duration-200 shadow-[3px_3px_0px_0px_rgba(0,0,0,1)] sm:shadow-[4px_4px_0px_0px_rgba(0,0,0,1)] hover:shadow-none hover:-translate-y-0.5 active:translate-y-0 bg-white border-zinc-900 text-zinc-900 hover:bg-zinc-100">
+    Go Back
+  </button>
+
+  <%# Modal Dialog %>
+  <dialog data-dialog-target="dialog"
+          class="fixed m-auto inset-0 z-50 w-[90vw] sm:w-[480px] max-w-[480px] p-0 bg-[#F8F5EC] text-zinc-900 shadow-[8px_8px_0px_0px_rgba(24,24,27,1)] border-[3px] border-zinc-900 outline-none backdrop:bg-zinc-900/50 backdrop:backdrop-blur-sm">
+
+    <%# Header %>
+    <div class="flex justify-between items-center px-4 py-3 bg-zinc-900 text-white border-b-[3px] border-zinc-900">
+      <span class="font-black text-sm uppercase tracking-widest">"CONFIRM"</span>
+      <button type="button" data-action="click->dialog#close" class="text-white hover:text-zinc-300 transition-colors">
+        <%= icon_tag "close", class: "text-xl" %>
+      </button>
+    </div>
+
+    <%# Content %>
+    <div class="p-6">
+      <div class="text-center mb-6">
+        <p class="font-black text-lg uppercase tracking-tight text-zinc-900 mb-2">
+          "GO BACK?"
+        </p>
+        <p class="text-sm text-zinc-600">
+          You are about to go back to <span class="font-bold text-zinc-900">"<%= previous_phase_name %>"</span>
+        </p>
+      </div>
+
+      <%# Warning about going back %>
+      <div class="mb-6 p-4 bg-amber-100 border-2 border-amber-500">
+        <div class="flex items-start gap-3">
+          <span class="w-8 h-8 flex items-center justify-center bg-amber-500 text-white font-black text-lg flex-shrink-0 border-2 border-amber-700">!</span>
+          <div>
+            <p class="font-bold text-sm uppercase tracking-wide text-amber-800 mb-1">
+              Participants will be redirected
+            </p>
+            <p class="text-sm text-amber-700">
+              All participants will be taken back to the previous phase.
+            </p>
+          </div>
+        </div>
+      </div>
+
+      <%# Action Buttons %>
+      <div class="flex gap-3 justify-end">
+        <button type="button"
+                data-action="click->dialog#close"
+                class="h-12 px-6 font-bold uppercase tracking-widest text-sm border-[2px] rounded-[1px] transition-all duration-200 shadow-[3px_3px_0px_0px_rgba(0,0,0,1)] hover:shadow-none bg-white border-zinc-900 text-zinc-900 hover:bg-zinc-100">
+          Cancel
+        </button>
+
+        <%= button_to retro_phase_back_path(@retro),
+              method: :post,
+              class: "h-12 px-6 font-bold uppercase tracking-widest text-sm border-[2px] rounded-[1px] transition-all duration-200 shadow-[3px_3px_0px_0px_rgba(0,0,0,1)] hover:shadow-none bg-zinc-900 border-zinc-900 text-white hover:bg-zinc-700" do %>
+          Yes, Go Back
+        <% end %>
+      </div>
+    </div>
+
+    <%# Footer Decoration %>
+    <div class="h-2 w-full bg-[repeating-linear-gradient(-45deg,#000,#000_2px,transparent_2px,transparent_4px)] opacity-10"></div>
+  </dialog>
+</div>

app/components/confirm_back_phase_modal_component.rb

@@ -0,0 +1,11 @@
+class ConfirmBackPhaseModalComponent < ApplicationComponent
+  def initialize(retro:)
+    @retro = retro
+  end
+
+  private
+
+  def previous_phase_name
+    @retro.previous_phase&.to_s&.tr("_", " ")&.upcase || "PREVIOUS PHASE"
+  end
+end

app/controllers/retros/phase_backs_controller.rb

@@ -0,0 +1,39 @@
+class Retros::PhaseBacksController < ApplicationController
+  include RetroAuthorization
+
+  before_action :set_retro
+  before_action :ensure_retro_admin
+
+  def create
+    @retro.back_phase!
+    broadcast_phase_redirect
+    redirect_to phase_path_for(@retro)
+  end
+
+  private
+
+  def set_retro
+    @retro = Current.account.retros.find(params[:retro_id])
+  end
+
+  def phase_path_for(retro)
+    case retro.phase.to_sym
+    when :waiting_room then retro_waiting_room_path(retro)
+    when :action_review then retro_action_review_path(retro)
+    when :brainstorming then retro_brainstorming_path(retro)
+    when :grouping then retro_grouping_path(retro)
+    when :voting then retro_voting_path(retro)
+    when :discussion then retro_discussion_path(retro)
+    when :complete then retro_complete_path(retro)
+    else retro_path(retro)
+    end
+  end
+
+  def broadcast_phase_redirect
+    redirect_url = phase_path_for(@retro)
+    Turbo::StreamsChannel.broadcast_stream_to(
+      @retro,
+      content: %(<turbo-stream action="redirect" url="#{redirect_url}"></turbo-stream>)
+    )
+  end
+end

app/models/retro.rb

@@ -45,6 +45,29 @@ def last_phase?
     phase.to_sym == PHASE_ORDER.last
   end
 
+  def back_phase!
+    current_index = PHASE_ORDER.index(phase.to_sym)
+    return if current_index.nil? || current_index <= PHASE_ORDER.index(:brainstorming)
+
+    from_phase = phase
+    prev_phase = PHASE_ORDER[current_index - 1]
+    update!(phase: prev_phase, highlighted_user_id: nil)
+    participants.update_all(finished: false)
+
+    record_event("retro.phase_changed", particulars: { from: from_phase, to: phase, direction: "back" })
+  end
+
+  def previous_phase
+    current_index = PHASE_ORDER.index(phase.to_sym)
+    return nil if current_index.nil? || current_index <= PHASE_ORDER.index(:brainstorming)
+    PHASE_ORDER[current_index - 1]
+  end
+
+  def can_go_back?
+    current_index = PHASE_ORDER.index(phase.to_sym)
+    current_index.present? && current_index > PHASE_ORDER.index(:brainstorming)
+  end
+
   def add_participant(user, role: :participant)
     participants.find_or_create_by!(user: user) do |p|
       p.role = role

app/views/layouts/retro.html.erb

@@ -106,6 +106,11 @@
             <% end %>
           <% end %>
 
+          <%# Go Back button with confirmation modal - admin only, when can go back %>
+          <% if @retro.admin?(Current.user) && @retro.can_go_back? %>
+            <%= render ConfirmBackPhaseModalComponent.new(retro: @retro) %>
+          <% end %>
+
           <%# Complete Phase button with confirmation modal - admin only, not shown on last phase %>
           <% if @retro.admin?(Current.user) && !@retro.last_phase? %>
             <%= render ConfirmPhaseModalComponent.new(retro: @retro) %>

config/brakeman.ignore

@@ -1,22 +1,5 @@
 {
-  "ignored_warnings": [
-    {
-      "warning_type": "Dynamic Render Path",
-      "warning_code": 15,
-      "fingerprint": "7c922d5793fec09548004f709c579bcec4f00b78bd5ff362bedb8d3148d39a9b",
-      "check_name": "Render",
-      "message": "Render path contains parameter value",
-      "file": "app/views/feedbacks/edit.html.erb",
-      "line": 53,
-      "link": "https://brakemanscanner.org/docs/warning_types/dynamic_render_path/",
-      "code": "render(action => ButtonComponent.new(:variant => :primary, :href => retro_path(Current.account.retros.find(params[:retro_id])), :data => ({ :controller => \"hotkey\", :action => \"keydown.ctrl+enter@document->hotkey#click keydown.meta+enter@document->hotkey#click\" })).with_content(\"Done\"), {})",
-      "render_path": null,
-      "location": null,
-      "user_input": "params[:retro_id]",
-      "confidence": "Weak",
-      "note": "False positive - this is rendering a ViewComponent with a safe href attribute, not a dynamic template path"
-    }
-  ],
-  "updated": "2025-12-13",
+  "ignored_warnings": [],
+  "updated": "2026-01-06",
   "brakeman_version": "7.1.1"
 }

config/routes.rb

@@ -24,6 +24,7 @@
       resource :finished, only: :update
       resource :finish_brainstorming, only: :create
       resource :phase_transition, only: :create
+      resource :phase_back, only: :create
       resources :feedback_groups, only: %i[create destroy] do
         delete :remove_feedback, on: :collection
       end

test/controllers/retros/phase_backs_controller_test.rb

@@ -0,0 +1,80 @@
+require "test_helper"
+
+class Retros::PhaseBacksControllerTest < ActionDispatch::IntegrationTest
+  setup do
+    @retro = retros(:one)
+    @admin_user = users(:one)
+    @member_user = users(:two)
+    # Add member as participant (non-admin)
+    @retro.add_participant(@member_user, role: :participant)
+  end
+
+  test "admin can go back from grouping to brainstorming" do
+    sign_in_as :one
+    @retro.update!(phase: :grouping)
+
+    post retro_phase_back_path(@retro)
+
+    assert_redirected_to retro_brainstorming_path(@retro)
+    assert_equal "brainstorming", @retro.reload.phase
+  end
+
+  test "admin can go back from voting to grouping" do
+    sign_in_as :one
+    @retro.update!(phase: :voting)
+
+    post retro_phase_back_path(@retro)
+
+    assert_redirected_to retro_grouping_path(@retro)
+    assert_equal "grouping", @retro.reload.phase
+  end
+
+  test "admin can go back from discussion to voting" do
+    sign_in_as :one
+    @retro.update!(phase: :discussion)
+
+    post retro_phase_back_path(@retro)
+
+    assert_redirected_to retro_voting_path(@retro)
+    assert_equal "voting", @retro.reload.phase
+  end
+
+  test "admin can go back from complete to discussion" do
+    sign_in_as :one
+    @retro.update!(phase: :complete)
+
+    post retro_phase_back_path(@retro)
+
+    assert_redirected_to retro_discussion_path(@retro)
+    assert_equal "discussion", @retro.reload.phase
+  end
+
+  test "non-admin participant cannot go back" do
+    sign_in_as :two
+    @retro.update!(phase: :grouping)
+
+    post retro_phase_back_path(@retro)
+
+    assert_redirected_to retros_path
+    assert_equal "grouping", @retro.reload.phase
+  end
+
+  test "unauthenticated user cannot go back" do
+    @retro.update!(phase: :grouping)
+
+    post retro_phase_back_path(@retro, script_name: nil)
+
+    assert_redirected_to session_menu_path(script_name: nil)
+    assert_equal "grouping", @retro.reload.phase
+  end
+
+  test "going back does not change phase when already at brainstorming" do
+    sign_in_as :one
+    @retro.update!(phase: :brainstorming)
+
+    post retro_phase_back_path(@retro)
+
+    assert_redirected_to retro_brainstorming_path(@retro)
+    assert_equal "brainstorming", @retro.reload.phase
+  end
+end