Close as working with default fallback label

app/controllers/bubbles/pops_controller.rb

@@ -2,7 +2,7 @@ class Bubbles::PopsController < ApplicationController
   include BubbleScoped, BucketScoped
 
   def create
-    @bubble.pop!(user: Current.user)
+    @bubble.pop!(user: Current.user, reason: params[:reason])
     redirect_to @bubble
   end
 

app/models/account.rb

@@ -13,6 +13,12 @@ class Account < ApplicationRecord
   has_many :workflows, dependent: :destroy
   has_many :stages, through: :workflows, class_name: "Workflow::Stage"
 
+  has_many :pop_reasons, dependent: :destroy, class_name: "Pop::Reason" do
+    def labels
+      pluck(:label).presence || [ Pop::Reason::FALLBACK_LABEL ]
+    end
+  end
+
   has_many :tags, dependent: :destroy
 
   has_many_attached :uploads

app/models/bubble/poppable.rb

@@ -40,10 +40,10 @@ module Bubble::Poppable
     pop&.created_at
   end
 
-  def pop!(user: Current.user)
+  def pop!(user: Current.user, reason: nil)
     unless popped?
       transaction do
-        create_pop!(user: user)
+        create_pop! user: user, reason: reason
         track_event :popped, creator: user
       end
     end

app/models/pop.rb

@@ -1,4 +1,8 @@
 class Pop < ApplicationRecord
   belongs_to :bubble, touch: true
   belongs_to :user, optional: true
+
+  def reason
+    super || Pop::Reason::FALLBACK_LABEL
+  end
 end

app/models/pop/reason.rb

@@ -0,0 +1,5 @@
+class Pop::Reason < ApplicationRecord
+  belongs_to :account
+
+  FALLBACK_LABEL = "Done"
+end

app/views/bubbles/_card.html.erb

@@ -51,7 +51,7 @@
 
     <% if bubble.popped? %>
       <div class="card__popped flex flex-column">
-        <span class="card__popped-title txt-uppercase"><%= bubble.popped_by.system? ? "Closed" : "Completed" %></span>
+        <span class="card__popped-title txt-uppercase"><%= bubble.popped_by.system? ? "Closed" : bubble.pop.reason %></span>
         <strong><%= bubble.popped_at.strftime("%b %d, %Y") %></strong>
         <span>by <span class="card__popped-by"><%= bubble.popped_by.name %></span></span>
       </div>

app/views/bubbles/_pop_toggle.html.erb

@@ -14,18 +14,13 @@
           aria-label="Close as…" aria-description="Close this card and select a reason.."
           data-dialog-target="dialog" data-action="turbo:before-cache@document->dialog#close">
         <strong class="popup__title margin-block-start-half pad-inline-half">Close as…</strong>
-        <%= button_to bucket_bubble_pop_path(bubble.bucket, bubble), class: "btn popup__item full-width", form_class: "full-width" do %>
-          <span class="overflow-ellipsis">Completed</span>
-        <% end %>
-        <%= button_to bucket_bubble_pop_path(bubble.bucket, bubble), class: "btn popup__item full-width", form_class: "full-width" do %>
-          <span class="overflow-ellipsis">Duplicate</span>
-        <% end %>
-        <%= button_to bucket_bubble_pop_path(bubble.bucket, bubble), class: "btn popup__item full-width", form_class: "full-width" do %>
-          <span class="overflow-ellipsis">Maybe later</span>
-        <% end %>
-        <%= button_to bucket_bubble_pop_path(bubble.bucket, bubble), class: "btn popup__item full-width", form_class: "full-width" do %>
-          <span class="overflow-ellipsis">Working as intended</span>
+
+        <% Current.account.pop_reasons.labels.each do |label| %>
+          <%= button_to bucket_bubble_pop_path(bubble.bucket, bubble, reason: label), class: "btn popup__item full-width", form_class: "full-width" do %>
+            <span class="overflow-ellipsis"><%= label %></span>
+          <% end %>
         <% end %>
+
         <% if bubble.doing? %>
           <hr class="separator--horizontal full-width flex-item-grow" style="--border-color: var(--color-subtle)" aria-hidden="true">
           <%= button_to bucket_bubble_engagement_path(bubble.bucket, bubble), method: :delete, class: "btn popup__item full-width" do %>

db/migrate/20250404083727_add_reason_to_pops.rb

@@ -0,0 +1,12 @@
+class AddReasonToPops < ActiveRecord::Migration[8.1]
+  def change
+    add_column :pops, :reason, :string
+
+    create_table :pop_reasons do |t|
+      t.references :account, index: true
+      t.string :label
+
+      t.timestamps
+    end
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[8.1].define(version: 2025_04_04_074315) do
+ActiveRecord::Schema[8.1].define(version: 2025_04_04_083727) do
   create_table "accesses", force: :cascade do |t|
     t.integer "bucket_id", null: false
     t.datetime "created_at", null: false
@@ -235,9 +235,18 @@ ActiveRecord::Schema[8.1].define(version: 2025_04_04_074315) do
     t.index ["user_id"], name: "index_pins_on_user_id"
   end
 
+  create_table "pop_reasons", force: :cascade do |t|
+    t.integer "account_id"
+    t.datetime "created_at", null: false
+    t.string "label"
+    t.datetime "updated_at", null: false
+    t.index ["account_id"], name: "index_pop_reasons_on_account_id"
+  end
+
   create_table "pops", force: :cascade do |t|
     t.integer "bubble_id", null: false
     t.datetime "created_at", null: false
+    t.string "reason"
     t.datetime "updated_at", null: false
     t.integer "user_id"
     t.index ["bubble_id"], name: "index_pops_on_bubble_id", unique: true

db/schema_cache.yml

@@ -468,6 +468,16 @@ columns:
     default_function:
     collation:
     comment:
+  - !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
+    auto_increment:
+    name: last_active_at
+    cast_type: *3
+    sql_type_metadata: *4
+    'null': true
+    default:
+    default_function:
+    collation:
+    comment:
   - !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
     auto_increment:
     name: stage_id
@@ -499,16 +509,6 @@ columns:
     collation:
     comment:
   - *9
-  - !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
-    auto_increment:
-    name: last_active_at
-    cast_type: *3
-    sql_type_metadata: *4
-    'null': true
-    default:
-    default_function:
-    collation:
-    comment:
   buckets:
   - *5
   - &32 !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
@@ -748,6 +748,30 @@ columns:
   - *6
   - *9
   - *30
+  pop_reasons:
+  - *5
+  - !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
+    auto_increment:
+    name: account_id
+    cast_type: *1
+    sql_type_metadata: *2
+    'null': true
+    default:
+    default_function:
+    collation:
+    comment:
+  - !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
+    auto_increment:
+    name: label
+    cast_type: *7
+    sql_type_metadata: *8
+    'null': true
+    default:
+    default_function:
+    collation:
+    comment:
+  - *6
+  - *9
   pops:
   - *5
   - *25
@@ -763,6 +787,16 @@ columns:
     default_function:
     collation:
     comment:
+  - !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
+    auto_increment:
+    name: reason
+    cast_type: *7
+    sql_type_metadata: *8
+    'null': true
+    default:
+    default_function:
+    collation:
+    comment:
   reactions:
   - *5
   - !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
@@ -963,6 +997,7 @@ primary_keys:
   messages: id
   notifications: id
   pins: id
+  pop_reasons: id
   pops: id
   reactions: id
   schema_migrations: version
@@ -998,6 +1033,7 @@ data_sources:
   messages: true
   notifications: true
   pins: true
+  pop_reasons: true
   pops: true
   reactions: true
   schema_migrations: true
@@ -1318,11 +1354,10 @@ indexes:
   bubbles:
   - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
     table: bubbles
-    name: index_bubbles_on_last_active_at_and_status
+    name: index_bubbles_on_stage_id
     unique: false
     columns:
-    - last_active_at
-    - status
+    - stage_id
     lengths: {}
     orders: {}
     opclasses: {}
@@ -1335,10 +1370,11 @@ indexes:
     valid: true
   - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
     table: bubbles
-    name: index_bubbles_on_stage_id
+    name: index_bubbles_on_last_active_at_and_status
     unique: false
     columns:
-    - stage_id
+    - last_active_at
+    - status
     lengths: {}
     orders: {}
     opclasses: {}
@@ -1802,6 +1838,23 @@ indexes:
     nulls_not_distinct:
     comment:
     valid: true
+  pop_reasons:
+  - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
+    table: pop_reasons
+    name: index_pop_reasons_on_account_id
+    unique: false
+    columns:
+    - account_id
+    lengths: {}
+    orders: {}
+    opclasses: {}
+    where:
+    type:
+    using:
+    include:
+    nulls_not_distinct:
+    comment:
+    valid: true
   pops:
   - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
     table: pops
@@ -2053,4 +2106,4 @@ indexes:
     nulls_not_distinct:
     comment:
     valid: true
-version: 20250404074315
+version: 20250404083727