From 793a9e3ecca56cc869a0e6317f33e529bd9fabef Mon Sep 17 00:00:00 2001 From: Jacob Cody Wimer Date: Tue, 11 Nov 2025 21:09:24 -0500 Subject: [PATCH] All links with a confirm now use turbo --- app/views/layouts/_tournament-navbar.html.erb | 14 +++++++------- app/views/mat_assignment_rules/index.html.erb | 2 +- app/views/schools/show.html.erb | 4 ++-- app/views/static_pages/my_tournaments.html.erb | 4 ++-- app/views/tournament_backups/index.html.erb | 4 ++-- app/views/tournaments/delegate.html.erb | 4 ++-- app/views/tournaments/index.html.erb | 3 +-- app/views/tournaments/school_delegate.html.erb | 2 +- app/views/tournaments/show.html.erb | 6 +++--- app/views/tournaments/teampointadjust.html.erb | 4 ++-- app/views/weights/show.html.erb | 4 ++-- 11 files changed, 25 insertions(+), 26 deletions(-) diff --git a/app/views/layouts/_tournament-navbar.html.erb b/app/views/layouts/_tournament-navbar.html.erb index d7f284d..df0814c 100644 --- a/app/views/layouts/_tournament-navbar.html.erb +++ b/app/views/layouts/_tournament-navbar.html.erb @@ -43,7 +43,7 @@
  • <%= link_to "Deduct Team Points" , "/tournaments/#{@tournament.id}/teampointadjust" %>
  • <%= link_to "View All Mat Assignment Rules", tournament_mat_assignment_rules_path(@tournament) %>
  • <%= link_to 'Manage Backups', tournament_tournament_backups_path(@tournament) %>
    • -
  • <%= link_to "Reset Bout Board", reset_bout_board_tournament_path(@tournament), data: { turbo_method: :post, turbo_confirm: "Are you sure you want to reset the bout board?" } %>
    • +
  • <%= link_to "Reset Bout Board", reset_bout_board_tournament_path(@tournament), data: { turbo_method: :post, turbo_confirm: "Are you sure you want to reset the bout board?" } %>
    • <% if can? :destroy, @tournament %>
  • <%= link_to "Tournament Delegation" , "/tournaments/#{@tournament.id}/delegate" %>
    • <% end %> @@ -55,13 +55,13 @@ <% end %> <% end %>
  • Time Savers
    • -
  • <%= link_to "Create Boys High School Weights (106-285)" , "/tournaments/#{@tournament.id}/create_custom_weights?customValue=#{Weight::HS_WEIGHT_CLASSES}",data: { confirm: 'Are you sure? This will delete all current weights.' } %>
    • -
  • <%= link_to "Create Girls High School Weights (100-235)" , "/tournaments/#{@tournament.id}/create_custom_weights?customValue=#{Weight::HS_GIRLS_WEIGHT_CLASSES}",data: { confirm: 'Are you sure? This will delete all current weights.' } %>
    • -
  • <%= link_to "Create Boys Middle School Weights (80-245)" , "/tournaments/#{@tournament.id}/create_custom_weights?customValue=#{Weight::MS_WEIGHT_CLASSES}",data: { confirm: 'Are you sure? This will delete all current weights.' } %>
    • -
  • <%= link_to "Create Girls Middle School Weights (72-235)" , "/tournaments/#{@tournament.id}/create_custom_weights?customValue=#{Weight::MS_GIRLS_WEIGHT_CLASSES}",data: { confirm: 'Are you sure? This will delete all current weights.' } %>
    • +
  • <%= link_to "Create Boys High School Weights (106-285)" , "/tournaments/#{@tournament.id}/create_custom_weights?customValue=#{Weight::HS_WEIGHT_CLASSES}",data: { turbo_method: :get, turbo_confirm: 'Are you sure? This will delete all current weights.' } %>
    • +
  • <%= link_to "Create Girls High School Weights (100-235)" , "/tournaments/#{@tournament.id}/create_custom_weights?customValue=#{Weight::HS_GIRLS_WEIGHT_CLASSES}",data: { turbo_method: :get, turbo_confirm: 'Are you sure? This will delete all current weights.' } %>
    • +
  • <%= link_to "Create Boys Middle School Weights (80-245)" , "/tournaments/#{@tournament.id}/create_custom_weights?customValue=#{Weight::MS_WEIGHT_CLASSES}",data: { turbo_method: :get, turbo_confirm: 'Are you sure? This will delete all current weights.' } %>
    • +
  • <%= link_to "Create Girls Middle School Weights (72-235)" , "/tournaments/#{@tournament.id}/create_custom_weights?customValue=#{Weight::MS_GIRLS_WEIGHT_CLASSES}",data: { turbo_method: :get, turbo_confirm: 'Are you sure? This will delete all current weights.' } %>
  • Tournament Actions
  • <%= link_to "Calculate Team Scores" , "/tournaments/#{@tournament.id}/calculate_team_scores", :method => :put %>
    • -
  • <%= link_to "Generate Brackets" , "/tournaments/#{@tournament.id}/generate_matches", data: { confirm: 'Are you sure? This will delete all current matches.' } %>
    • +
  • <%= link_to "Generate Brackets" , "/tournaments/#{@tournament.id}/generate_matches", data: { turbo_method: :get, turbo_confirm: 'Are you sure? This will delete all current matches.' } %>
  • <%= link_to "Export Data" , "/tournaments/#{@tournament.id}/export?print=true", target: :_blank %>
    • <% end %> @@ -69,4 +69,4 @@ -<% end %> +<% end %> diff --git a/app/views/mat_assignment_rules/index.html.erb b/app/views/mat_assignment_rules/index.html.erb index 4b54150..934ca82 100644 --- a/app/views/mat_assignment_rules/index.html.erb +++ b/app/views/mat_assignment_rules/index.html.erb @@ -23,7 +23,7 @@ <%= Array(rule.rounds).join(", ") %> <%= link_to '', edit_tournament_mat_assignment_rule_path(@tournament, rule), class: "fas fa-edit" %> - <%= link_to '', tournament_mat_assignment_rule_path(@tournament, rule), method: :delete, data: { confirm: "Are you sure?" }, class: "fas fa-trash-alt" %> + <%= link_to '', tournament_mat_assignment_rule_path(@tournament, rule), data: { turbo_method: :delete, turbo_confirm: "Are you sure?" }, class: "fas fa-trash-alt" %> <% end %> diff --git a/app/views/schools/show.html.erb b/app/views/schools/show.html.erb index 25f48fd..bba365f 100644 --- a/app/views/schools/show.html.erb +++ b/app/views/schools/show.html.erb @@ -77,7 +77,7 @@ <% delete_wrestler_path_with_key += "?school_permission_key=#{params[:school_permission_key]}" if params[:school_permission_key].present? %> <%= link_to '', edit_wrestler_path_with_key, class: "fas fa-edit" %> - <%= link_to '', delete_wrestler_path_with_key, method: :delete, data: { confirm: "Are you sure you want to delete #{wrestler.name}? This will delete all of his matches." }, class: "fas fa-trash-alt" %> + <%= link_to '', delete_wrestler_path_with_key, data: { turbo_method: :delete, turbo_confirm: "Are you sure you want to delete #{wrestler.name}? This will delete all of his matches." }, class: "fas fa-trash-alt" %> <% end %> @@ -102,7 +102,7 @@ <% delete_wrestler_path_with_key += "?school_permission_key=#{params[:school_permission_key]}" if params[:school_permission_key].present? %> <%= link_to '', edit_wrestler_path_with_key, class: "fas fa-edit" %> - <%= link_to '', delete_wrestler_path_with_key, method: :delete, data: { confirm: "Are you sure you want to delete #{wrestler.name}? This will delete all of his matches." }, class: "fas fa-trash-alt" %> + <%= link_to '', delete_wrestler_path_with_key, data: { turbo_method: :delete, turbo_confirm: "Are you sure you want to delete #{wrestler.name}? This will delete all of his matches." }, class: "fas fa-trash-alt" %> <% end %> diff --git a/app/views/static_pages/my_tournaments.html.erb b/app/views/static_pages/my_tournaments.html.erb index 0760544..020549d 100644 --- a/app/views/static_pages/my_tournaments.html.erb +++ b/app/views/static_pages/my_tournaments.html.erb @@ -24,7 +24,7 @@ <%= link_to '', edit_tournament_path(tournament), :class=>"fas fa-edit" %> <% if can? :destroy, tournament %> - <%= link_to '', tournament, method: :delete, data: { confirm: "Are you sure you want to delete #{tournament.name}?" }, :class=>"fas fa-trash-alt" %> + <%= link_to '', tournament, data: { turbo_method: :delete, turbo_confirm: "Are you sure you want to delete #{tournament.name}?" }, :class=>"fas fa-trash-alt" %> <% end %> <% end %> @@ -55,4 +55,4 @@ <% end %> -<% end %> \ No newline at end of file +<% end %> diff --git a/app/views/tournament_backups/index.html.erb b/app/views/tournament_backups/index.html.erb index c3fb464..ca28643 100644 --- a/app/views/tournament_backups/index.html.erb +++ b/app/views/tournament_backups/index.html.erb @@ -19,8 +19,8 @@ and will also delete all of your current data. It's best to use the create backu <%= backup.backup_reason.presence || 'No reason provided' %> - <%= link_to '', restore_tournament_tournament_backup_path(@tournament, backup), method: :post, data: { confirm: "This will restore the backup from #{backup.created_at.strftime('%Y-%m-%d %H:%M:%S')}. It will delete all current data from the tournament in order to restore the backup." }, class: 'fas fa-undo-alt text-warning', title: 'Restore Backup' %> - <%= link_to '', tournament_tournament_backup_path(@tournament, backup), method: :delete, data: { confirm: 'Are you sure you want to delete this backup?' }, class: 'fas fa-trash-alt', title: 'Delete Backup' %> + <%= link_to '', restore_tournament_tournament_backup_path(@tournament, backup), data: { turbo_method: :post, turbo_confirm: "This will restore the backup from #{backup.created_at.strftime('%Y-%m-%d %H:%M:%S')}. It will delete all current data from the tournament in order to restore the backup." }, class: 'fas fa-undo-alt text-warning', title: 'Restore Backup' %> + <%= link_to '', tournament_tournament_backup_path(@tournament, backup), data: { turbo_method: :delete, turbo_confirm: 'Are you sure you want to delete this backup?' }, class: 'fas fa-trash-alt', title: 'Delete Backup' %> <% end %> diff --git a/app/views/tournaments/delegate.html.erb b/app/views/tournaments/delegate.html.erb index 75fcd09..7be566f 100644 --- a/app/views/tournaments/delegate.html.erb +++ b/app/views/tournaments/delegate.html.erb @@ -42,10 +42,10 @@ <% @users_delegates.each do |delegate| %> <%= delegate.user.email %> - <%= link_to 'Remove permissions', "/tournaments/#{@tournament.id}/#{delegate.id}/remove_delegate", method: :delete, confirm: 'Are you sure?', :class=>"btn btn-danger btn-sm" %> + <%= link_to 'Remove permissions', "/tournaments/#{@tournament.id}/#{delegate.id}/remove_delegate", data: { turbo_method: :delete, turbo_confirm: 'Are you sure?' }, :class=>"btn btn-danger btn-sm" %> <% end %> -<% end %> \ No newline at end of file +<% end %> diff --git a/app/views/tournaments/index.html.erb b/app/views/tournaments/index.html.erb index 7389526..7cbfb7b 100644 --- a/app/views/tournaments/index.html.erb +++ b/app/views/tournaments/index.html.erb @@ -29,7 +29,7 @@ <% if can? :manage, tournament %> <%= link_to '', edit_tournament_path(tournament), :class=>"fas fa-edit" %> <% if can? :destroy, tournament %> - <%= link_to '', tournament, method: :delete, data: { confirm: "Are you sure you want to delete #{tournament.name}?" }, :class=>"fas fa-trash-alt" %> + <%= link_to '', tournament, data: { turbo_method: :delete, turbo_confirm: "Are you sure you want to delete #{tournament.name}?" }, :class=>"fas fa-trash-alt" %> <% end %> <% end %> @@ -83,4 +83,3 @@ Showing <%= start_index %> - <%= end_index %> of <%= @total_count %> tournaments

    <% end %> - diff --git a/app/views/tournaments/school_delegate.html.erb b/app/views/tournaments/school_delegate.html.erb index e3af2f2..f772cd5 100644 --- a/app/views/tournaments/school_delegate.html.erb +++ b/app/views/tournaments/school_delegate.html.erb @@ -78,7 +78,7 @@ <%= delegate.user.email %> <%= delegate.school.name %> - <%= link_to 'Remove permissions', "/tournaments/#{@tournament.id}/#{delegate.id}/remove_school_delegate", method: :delete, confirm: 'Are you sure?', :class=>"btn btn-danger btn-sm" %> + <%= link_to 'Remove permissions', "/tournaments/#{@tournament.id}/#{delegate.id}/remove_school_delegate", data: { turbo_method: :delete, turbo_confirm: 'Are you sure?' }, :class=>"btn btn-danger btn-sm" %> <% end %> diff --git a/app/views/tournaments/show.html.erb b/app/views/tournaments/show.html.erb index 6f5760d..7b5cc0a 100644 --- a/app/views/tournaments/show.html.erb +++ b/app/views/tournaments/show.html.erb @@ -72,7 +72,7 @@ <% if can? :manage, school %> <%= link_to '', edit_school_path(school), :class=>"fas fa-edit" %> <% if can? :manage, @tournament %> - <%= link_to '', school, method: :delete, data: { confirm: "Are you sure you want to delete #{school.name}?" }, :class=>"fas fa-trash-alt" %> + <%= link_to '', school, data: { turbo_method: :delete, turbo_confirm: "Are you sure you want to delete #{school.name}?" }, :class=>"fas fa-trash-alt" %> <% end %> <% end %> @@ -106,7 +106,7 @@ <% if can? :manage, @tournament %> <%= link_to '', edit_weight_path(weight), :class=>"fas fa-edit" %> - <%= link_to '', weight, method: :delete, data: { confirm: "Are you sure you want to delete the #{weight.max} weight class?" }, :class=>"fas fa-trash-alt" %> + <%= link_to '', weight, data: { turbo_method: :delete, turbo_confirm: "Are you sure you want to delete the #{weight.max} weight class?" }, :class=>"fas fa-trash-alt" %> <% end %> @@ -130,7 +130,7 @@ <%= link_to "Mat #{mat.name}", mat %> <% if can? :manage, @tournament %> - <%= link_to '', mat, method: :delete, data: { confirm: "Are you sure you want to delete Mat #{mat.name}?" }, :class=>"fas fa-trash-alt" %> + <%= link_to '', mat, data: { turbo_method: :delete, turbo_confirm: "Are you sure you want to delete Mat #{mat.name}?" }, :class=>"fas fa-trash-alt" %> <%= link_to '', "/mats/#{mat.id}/assign_next_match", method: :post, :class=>"fas fa-solid fa-arrow-right" %> <% end %> diff --git a/app/views/tournaments/teampointadjust.html.erb b/app/views/tournaments/teampointadjust.html.erb index 0a0978b..60e4bad 100644 --- a/app/views/tournaments/teampointadjust.html.erb +++ b/app/views/tournaments/teampointadjust.html.erb @@ -47,10 +47,10 @@ <% end %> <%= point_adjustment.points %> - <%= link_to 'Remove Point Adjustment', "/tournaments/#{@tournament.id}/#{point_adjustment.id}/remove_teampointadjust", method: :delete, confirm: 'Are you sure?', :class=>"btn btn-danger btn-sm" %> + <%= link_to 'Remove Point Adjustment', "/tournaments/#{@tournament.id}/#{point_adjustment.id}/remove_teampointadjust", data: { turbo_method: :delete, turbo_confirm: 'Are you sure?' }, :class=>"btn btn-danger btn-sm" %> <% end %> -<% end %> \ No newline at end of file +<% end %> diff --git a/app/views/weights/show.html.erb b/app/views/weights/show.html.erb index 0b237ec..a36b927 100644 --- a/app/views/weights/show.html.erb +++ b/app/views/weights/show.html.erb @@ -35,7 +35,7 @@ <% end %> <% if can? :manage, @tournament %> - <%= link_to '', wrestler, method: :delete, data: { confirm: "Are you sure you want to delete #{wrestler.name}? THIS WILL DELETE ALL MATCHES." } , :class=>"fas fa-trash-alt" %> + <%= link_to '', wrestler, data: { turbo_method: :delete, turbo_confirm: "Are you sure you want to delete #{wrestler.name}? THIS WILL DELETE ALL MATCHES." } , :class=>"fas fa-trash-alt" %> <% end %> @@ -81,4 +81,4 @@ -<% end %> \ No newline at end of file +<% end %>