Use Card number as the model param

instead of the UUID `id`
This commit is contained in:
Mike Dalessio
2025-11-14 16:31:11 -05:00
parent 2a1817d902
commit f5305b2fe6
7 changed files with 10 additions and 6 deletions
+1 -1
View File
@@ -16,6 +16,6 @@ class Cards::BoardsController < ApplicationController
private
def set_card
@card = Current.user.accessible_cards.find(params[:card_id])
@card = Current.user.accessible_cards.find_by!(number: params[:card_id])
end
end
+1 -1
View File
@@ -1,6 +1,6 @@
class Cards::ColumnsController < ApplicationController
def edit
@card = Current.user.accessible_cards.find(params[:card_id])
@card = Current.user.accessible_cards.find_by!(number: params[:card_id])
@columns = @card.board.columns.sorted
fresh_when etag: [ @card, @columns ]
+1 -1
View File
@@ -39,7 +39,7 @@ class CardsController < ApplicationController
end
def set_card
@card = Current.user.accessible_cards.find params[:id]
@card = Current.user.accessible_cards.find_by!(number: params[:id])
end
def ensure_permission_to_administer_card
+1 -1
View File
@@ -7,7 +7,7 @@ module CardScoped
private
def set_card
@card = Current.user.accessible_cards.find(params[:card_id])
@card = Current.user.accessible_cards.find_by!(number: params[:card_id])
end
def set_board
+1 -1
View File
@@ -30,7 +30,7 @@ class Prompts::CardsController < ApplicationController
end
def prepending_exact_matches_by_id(cards)
if card_by_id = Current.user.accessible_cards.find_by_id(params[:filter])
if card_by_id = Current.user.accessible_cards.find_by(number: params[:filter])
[ card_by_id ] + cards
else
cards
+1 -1
View File
@@ -11,7 +11,7 @@ class Public::BaseController < ApplicationController
end
def set_card
@card = @board.cards.find(params[:id]) if params[:board_id] && params[:id]
@card = @board.cards.find_by!(number: params[:id]) if params[:board_id] && params[:id]
end
def set_public_cache_expiration
+4
View File
@@ -47,6 +47,10 @@ class Card < ApplicationRecord
self
end
def to_param
number.to_s
end
def move_to(new_board)
transaction do
card.update!(board: new_board)