-
+ <%= render "cards/golden_toggle", card: @card if @card.doing? %>
<%= render "cards/image", card: @card %>
<%= button_to collection_card_path(@card.collection, @card),
diff --git a/config/routes.rb b/config/routes.rb
index cf2b5fe76..70de37a31 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -22,6 +22,7 @@ Rails.application.routes.draw do
resource :reading
resource :recover
resource :watch
+ resource :goldness
resources :assignments
resources :boosts
diff --git a/db/migrate/20250410141409_create_card_goldnesses.rb b/db/migrate/20250410141409_create_card_goldnesses.rb
new file mode 100644
index 000000000..f8268c243
--- /dev/null
+++ b/db/migrate/20250410141409_create_card_goldnesses.rb
@@ -0,0 +1,9 @@
+class CreateCardGoldnesses < ActiveRecord::Migration[8.1]
+ def change
+ create_table :card_goldnesses do |t|
+ t.references :card, null: false, foreign_key: true, index: { unique: true }
+
+ t.timestamps
+ end
+ end
+end
diff --git a/db/schema.rb b/db/schema.rb
index b7e50d935..4fa6fa273 100644
--- a/db/schema.rb
+++ b/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_09_201424) do
+ActiveRecord::Schema[8.1].define(version: 2025_04_10_141409) do
create_table "accesses", force: :cascade do |t|
t.integer "collection_id", null: false
t.datetime "created_at", null: false
@@ -102,6 +102,13 @@ ActiveRecord::Schema[8.1].define(version: 2025_04_09_201424) do
t.index ["card_id"], name: "index_card_engagements_on_card_id"
end
+ create_table "card_goldnesses", force: :cascade do |t|
+ t.integer "card_id", null: false
+ t.datetime "created_at", null: false
+ t.datetime "updated_at", null: false
+ t.index ["card_id"], name: "index_card_goldnesses_on_card_id", unique: true
+ end
+
create_table "cards", force: :cascade do |t|
t.float "activity_score", default: 0.0, null: false
t.datetime "activity_score_at"
@@ -333,6 +340,7 @@ ActiveRecord::Schema[8.1].define(version: 2025_04_09_201424) do
add_foreign_key "active_storage_attachments", "active_storage_blobs", column: "blob_id"
add_foreign_key "active_storage_variant_records", "active_storage_blobs", column: "blob_id"
+ add_foreign_key "card_goldnesses", "cards"
add_foreign_key "cards", "workflow_stages", column: "stage_id"
add_foreign_key "closures", "cards"
add_foreign_key "closures", "users"
diff --git a/db/schema_cache.yml b/db/schema_cache.yml
index 592648228..3233fb2f6 100644
--- a/db/schema_cache.yml
+++ b/db/schema_cache.yml
@@ -21,7 +21,7 @@ columns:
default_function:
collation:
comment:
- - &23 !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
+ - &24 !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
auto_increment:
name: collection_id
cast_type: *1
@@ -341,7 +341,7 @@ columns:
- *5
- *19
- *20
- - &24 !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
+ - &21 !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
auto_increment:
name: card_id
cast_type: *1
@@ -367,16 +367,21 @@ columns:
comment:
- *6
- *9
+ card_goldnesses:
+ - *5
+ - *21
+ - *6
+ - *9
cards:
- *5
- !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
auto_increment:
name: activity_score
- cast_type: &21 !ruby/object:ActiveModel::Type::Float
+ cast_type: &22 !ruby/object:ActiveModel::Type::Float
precision:
scale:
limit:
- sql_type_metadata: &22 !ruby/object:ActiveRecord::ConnectionAdapters::SqlTypeMetadata
+ sql_type_metadata: &23 !ruby/object:ActiveRecord::ConnectionAdapters::SqlTypeMetadata
sql_type: float
type: :float
limit:
@@ -400,8 +405,8 @@ columns:
- !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
auto_increment:
name: activity_score_order
- cast_type: *21
- sql_type_metadata: *22
+ cast_type: *22
+ sql_type_metadata: *23
'null': false
default: 0.0
default_function:
@@ -417,7 +422,7 @@ columns:
default_function:
collation:
comment:
- - *23
+ - *24
- !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
auto_increment:
name: comments_count
@@ -525,7 +530,7 @@ columns:
- *9
closures:
- *5
- - *24
+ - *21
- *6
- !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
auto_increment:
@@ -593,7 +598,7 @@ columns:
collation:
comment:
collections_filters:
- - *23
+ - *24
- *18
comments:
- *5
@@ -619,7 +624,7 @@ columns:
default_function:
collation:
comment:
- - *24
+ - *21
- *6
- *25
- !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
@@ -712,7 +717,7 @@ columns:
comment:
messages:
- *5
- - *24
+ - *21
- *6
- !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
auto_increment:
@@ -737,7 +742,7 @@ columns:
- *9
notifications:
- *5
- - *24
+ - *21
- *6
- !ruby/object:ActiveRecord::ConnectionAdapters::SQLite3::Column
auto_increment:
@@ -783,7 +788,7 @@ columns:
- *30
pins:
- *5
- - *24
+ - *21
- *6
- *9
- *30
@@ -869,7 +874,7 @@ columns:
- *30
taggings:
- *5
- - *24
+ - *21
- *6
- *31
- *9
@@ -927,7 +932,7 @@ columns:
- *9
watches:
- *5
- - *24
+ - *21
- *6
- *9
- *30
@@ -984,6 +989,7 @@ primary_keys:
assigners_filters:
assignments: id
card_engagements: id
+ card_goldnesses: id
cards: id
closure_reasons: id
closures: id
@@ -1020,6 +1026,7 @@ data_sources:
assigners_filters: true
assignments: true
card_engagements: true
+ card_goldnesses: true
cards: true
closure_reasons: true
closures: true
@@ -1048,10 +1055,9 @@ indexes:
accesses:
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: accesses
- name: index_accesses_on_collection_id_and_user_id
- unique: true
+ name: index_accesses_on_user_id
+ unique: false
columns:
- - collection_id
- user_id
lengths: {}
orders: {}
@@ -1081,9 +1087,10 @@ indexes:
valid: true
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: accesses
- name: index_accesses_on_user_id
- unique: false
+ name: index_accesses_on_collection_id_and_user_id
+ unique: true
columns:
+ - collection_id
- user_id
lengths: {}
orders: {}
@@ -1303,10 +1310,9 @@ indexes:
assignments:
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: assignments
- name: index_assignments_on_assignee_id_and_card_id
- unique: true
+ name: index_assignments_on_card_id
+ unique: false
columns:
- - assignee_id
- card_id
lengths: {}
orders: {}
@@ -1320,9 +1326,10 @@ indexes:
valid: true
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: assignments
- name: index_assignments_on_card_id
- unique: false
+ name: index_assignments_on_assignee_id_and_card_id
+ unique: true
columns:
+ - assignee_id
- card_id
lengths: {}
orders: {}
@@ -1351,7 +1358,57 @@ indexes:
nulls_not_distinct:
comment:
valid: true
+ card_goldnesses:
+ - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
+ table: card_goldnesses
+ name: index_card_goldnesses_on_card_id
+ unique: true
+ columns:
+ - card_id
+ lengths: {}
+ orders: {}
+ opclasses: {}
+ where:
+ type:
+ using:
+ include:
+ nulls_not_distinct:
+ comment:
+ valid: true
cards:
+ - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
+ table: cards
+ name: index_cards_on_stage_id
+ unique: false
+ columns:
+ - stage_id
+ lengths: {}
+ orders: {}
+ opclasses: {}
+ where:
+ type:
+ using:
+ include:
+ nulls_not_distinct:
+ comment:
+ valid: true
+ - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
+ table: cards
+ name: index_cards_on_last_active_at_and_status
+ unique: false
+ columns:
+ - last_active_at
+ - status
+ lengths: {}
+ orders: {}
+ opclasses: {}
+ where:
+ type:
+ using:
+ include:
+ nulls_not_distinct:
+ comment:
+ valid: true
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: cards
name: index_cards_on_collection_id
@@ -1384,39 +1441,6 @@ indexes:
nulls_not_distinct:
comment:
valid: true
- - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
- table: cards
- name: index_cards_on_last_active_at_and_status
- unique: false
- columns:
- - last_active_at
- - status
- lengths: {}
- orders: {}
- opclasses: {}
- where:
- type:
- using:
- include:
- nulls_not_distinct:
- comment:
- valid: true
- - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
- table: cards
- name: index_cards_on_stage_id
- unique: false
- columns:
- - stage_id
- lengths: {}
- orders: {}
- opclasses: {}
- where:
- type:
- using:
- include:
- nulls_not_distinct:
- comment:
- valid: true
closure_reasons:
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: closure_reasons
@@ -1435,6 +1459,22 @@ indexes:
comment:
valid: true
closures:
+ - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
+ table: closures
+ name: index_closures_on_user_id
+ unique: false
+ columns:
+ - user_id
+ lengths: {}
+ orders: {}
+ opclasses: {}
+ where:
+ type:
+ using:
+ include:
+ nulls_not_distinct:
+ comment:
+ valid: true
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: closures
name: index_closures_on_card_id
@@ -1468,29 +1508,13 @@ indexes:
nulls_not_distinct:
comment:
valid: true
- - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
- table: closures
- name: index_closures_on_user_id
- unique: false
- columns:
- - user_id
- lengths: {}
- orders: {}
- opclasses: {}
- where:
- type:
- using:
- include:
- nulls_not_distinct:
- comment:
- valid: true
collections:
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: collections
- name: index_collections_on_account_id
+ name: index_collections_on_workflow_id
unique: false
columns:
- - account_id
+ - workflow_id
lengths: {}
orders: {}
opclasses: {}
@@ -1519,10 +1543,10 @@ indexes:
valid: true
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: collections
- name: index_collections_on_workflow_id
+ name: index_collections_on_account_id
unique: false
columns:
- - workflow_id
+ - account_id
lengths: {}
orders: {}
opclasses: {}
@@ -1536,10 +1560,10 @@ indexes:
collections_filters:
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: collections_filters
- name: index_collections_filters_on_collection_id
+ name: index_collections_filters_on_filter_id
unique: false
columns:
- - collection_id
+ - filter_id
lengths: {}
orders: {}
opclasses: {}
@@ -1552,10 +1576,10 @@ indexes:
valid: true
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: collections_filters
- name: index_collections_filters_on_filter_id
+ name: index_collections_filters_on_collection_id
unique: false
columns:
- - filter_id
+ - collection_id
lengths: {}
orders: {}
opclasses: {}
@@ -1602,22 +1626,6 @@ indexes:
valid: true
event_summaries: []
events:
- - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
- table: events
- name: index_events_on_card_id
- unique: false
- columns:
- - card_id
- lengths: {}
- orders: {}
- opclasses: {}
- where:
- type:
- using:
- include:
- nulls_not_distinct:
- comment:
- valid: true
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: events
name: index_events_on_summary_id_and_action
@@ -1651,6 +1659,22 @@ indexes:
nulls_not_distinct:
comment:
valid: true
+ - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
+ table: events
+ name: index_events_on_card_id
+ unique: false
+ columns:
+ - card_id
+ lengths: {}
+ orders: {}
+ opclasses: {}
+ where:
+ type:
+ using:
+ include:
+ nulls_not_distinct:
+ comment:
+ valid: true
filters:
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: filters
@@ -1736,22 +1760,6 @@ indexes:
comment:
valid: true
messages:
- - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
- table: messages
- name: index_messages_on_card_id
- unique: false
- columns:
- - card_id
- lengths: {}
- orders: {}
- opclasses: {}
- where:
- type:
- using:
- include:
- nulls_not_distinct:
- comment:
- valid: true
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: messages
name: index_messages_on_messageable
@@ -1769,10 +1777,9 @@ indexes:
nulls_not_distinct:
comment:
valid: true
- notifications:
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
- table: notifications
- name: index_notifications_on_card_id
+ table: messages
+ name: index_messages_on_card_id
unique: false
columns:
- card_id
@@ -1786,6 +1793,7 @@ indexes:
nulls_not_distinct:
comment:
valid: true
+ notifications:
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: notifications
name: index_notifications_on_user_id
@@ -1855,13 +1863,28 @@ indexes:
nulls_not_distinct:
comment:
valid: true
+ - !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
+ table: notifications
+ name: index_notifications_on_card_id
+ unique: false
+ columns:
+ - card_id
+ lengths: {}
+ orders: {}
+ opclasses: {}
+ where:
+ type:
+ using:
+ include:
+ nulls_not_distinct:
+ comment:
+ valid: true
pins:
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: pins
- name: index_pins_on_card_id_and_user_id
- unique: true
+ name: index_pins_on_user_id
+ unique: false
columns:
- - card_id
- user_id
lengths: {}
orders: {}
@@ -1891,9 +1914,10 @@ indexes:
valid: true
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: pins
- name: index_pins_on_user_id
- unique: false
+ name: index_pins_on_card_id_and_user_id
+ unique: true
columns:
+ - card_id
- user_id
lengths: {}
orders: {}
@@ -1959,10 +1983,9 @@ indexes:
taggings:
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: taggings
- name: index_taggings_on_card_id_and_tag_id
- unique: true
+ name: index_taggings_on_tag_id
+ unique: false
columns:
- - card_id
- tag_id
lengths: {}
orders: {}
@@ -1976,9 +1999,10 @@ indexes:
valid: true
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: taggings
- name: index_taggings_on_tag_id
- unique: false
+ name: index_taggings_on_card_id_and_tag_id
+ unique: true
columns:
+ - card_id
- tag_id
lengths: {}
orders: {}
@@ -2076,10 +2100,10 @@ indexes:
watches:
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: watches
- name: index_watches_on_card_id
+ name: index_watches_on_user_id
unique: false
columns:
- - card_id
+ - user_id
lengths: {}
orders: {}
opclasses: {}
@@ -2092,10 +2116,10 @@ indexes:
valid: true
- !ruby/object:ActiveRecord::ConnectionAdapters::IndexDefinition
table: watches
- name: index_watches_on_user_id
+ name: index_watches_on_card_id
unique: false
columns:
- - user_id
+ - card_id
lengths: {}
orders: {}
opclasses: {}
@@ -2140,4 +2164,4 @@ indexes:
nulls_not_distinct:
comment:
valid: true
-version: 20250409201424
+version: 20250410141409
diff --git a/lib/rails_ext/prepend_order.rb b/lib/rails_ext/prepend_order.rb
new file mode 100644
index 000000000..b7eb995fa
--- /dev/null
+++ b/lib/rails_ext/prepend_order.rb
@@ -0,0 +1,16 @@
+module ActiveRecordRelationPrependOrder
+ extend ActiveSupport::Concern
+
+ included do
+ def prepend_order(*args)
+ new_orders = args.flatten.map { |arg| arg.is_a?(String) ? arg : arg.to_sql }
+
+ spawn.tap do |relation|
+ relation.order_values = new_orders + order_values
+ end
+ end
+ end
+end
+
+ActiveRecord::Relation.include(ActiveRecordRelationPrependOrder)
+ActiveRecord::AssociationRelation.include(ActiveRecordRelationPrependOrder)
diff --git a/test/controllers/cards/goldnesses_controller_test.rb b/test/controllers/cards/goldnesses_controller_test.rb
new file mode 100644
index 000000000..081a6ec61
--- /dev/null
+++ b/test/controllers/cards/goldnesses_controller_test.rb
@@ -0,0 +1,27 @@
+require "test_helper"
+
+class Cards::GoldnessesControllerTest < ActionDispatch::IntegrationTest
+ setup do
+ sign_in_as :kevin
+ end
+
+ test "create" do
+ card = cards(:text)
+
+ assert_changes -> { card.reload.golden? }, from: false, to: true do
+ post card_goldness_url(card)
+ end
+
+ assert_redirected_to collection_card_url(card.collection, card)
+ end
+
+ test "destroy" do
+ card = cards(:logo)
+
+ assert_changes -> { card.reload.golden? }, from: true, to: false do
+ delete card_goldness_url(card)
+ end
+
+ assert_redirected_to collection_card_url(card.collection, card)
+ end
+end
diff --git a/test/fixtures/card/engagements.yml b/test/fixtures/card/engagements.yml
index f15900c8b..77657385b 100644
--- a/test/fixtures/card/engagements.yml
+++ b/test/fixtures/card/engagements.yml
@@ -1,2 +1,2 @@
-logo_engagement:
+logo:
card: logo
diff --git a/test/fixtures/card/goldnesses.yml b/test/fixtures/card/goldnesses.yml
new file mode 100644
index 000000000..77657385b
--- /dev/null
+++ b/test/fixtures/card/goldnesses.yml
@@ -0,0 +1,2 @@
+logo:
+ card: logo
diff --git a/test/models/card/golden_test.rb b/test/models/card/golden_test.rb
new file mode 100644
index 000000000..be501c754
--- /dev/null
+++ b/test/models/card/golden_test.rb
@@ -0,0 +1,27 @@
+require "test_helper"
+
+class Card::GoldenTest < ActiveSupport::TestCase
+ setup do
+ Current.session = sessions(:david)
+ end
+
+ test "check whether a card is golden" do
+ assert cards(:logo).golden?
+ assert_not cards(:text).golden?
+ end
+
+ test "promote and demote from golden" do
+ assert_changes -> { cards(:text).reload.golden? }, to: true do
+ cards(:text).gild
+ end
+
+ assert_changes -> { cards(:logo).reload.golden? }, to: false do
+ cards(:logo).ungild
+ end
+ end
+
+ test "scopes" do
+ assert_includes Card.golden, cards(:logo)
+ assert_not_includes Card.golden, cards(:text)
+ end
+end