From 3fcd4f80aeeb4dcbf04fd1c2809b176ce2f3a28e Mon Sep 17 00:00:00 2001 From: Jeremy Walker Date: Wed, 15 Apr 2026 11:10:55 +0900 Subject: [PATCH 1/2] Remove max-width from .c-perk logo Co-Authored-By: Claude Opus 4.6 (1M context) --- app/css/components/perk.css | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/css/components/perk.css b/app/css/components/perk.css index 55b70700e2..abb0db3141 100644 --- a/app/css/components/perk.css +++ b/app/css/components/perk.css @@ -4,7 +4,7 @@ @apply flex flex-col items-stretch; .logo { - @apply h-[40px] max-w-[190px]; + @apply h-[40px]; @apply mb-12; @apply self-start; } From 32ff18c07a7bbcca69a50d9e2c6ff4f1cfa59ebb Mon Sep 17 00:00:00 2001 From: Jeremy Walker Date: Wed, 15 Apr 2026 11:17:08 +0900 Subject: [PATCH 2/2] Remove admin guard from advert click logging Co-Authored-By: Claude Opus 4.6 (1M context) --- app/commands/partner/log_advert_click.rb | 8 -------- test/commands/partner/log_advert_click_test.rb | 10 ---------- 2 files changed, 18 deletions(-) diff --git a/app/commands/partner/log_advert_click.rb b/app/commands/partner/log_advert_click.rb index 6ab6ac7422..d124695395 100644 --- a/app/commands/partner/log_advert_click.rb +++ b/app/commands/partner/log_advert_click.rb @@ -7,18 +7,10 @@ class LogAdvertClick initialize_with :advert, :user, :clicked_at, :impression_uuid def call - return unless valid_click? - Advert.where(id: advert.id).update_all('num_clicks = num_clicks + 1') end private - def valid_click? - return false if user&.admin? - - true - end - def doc { advert_id: advert.id, diff --git a/test/commands/partner/log_advert_click_test.rb b/test/commands/partner/log_advert_click_test.rb index 8131ea2202..bd5b4d7d27 100644 --- a/test/commands/partner/log_advert_click_test.rb +++ b/test/commands/partner/log_advert_click_test.rb @@ -9,14 +9,4 @@ class Partner::LogAdvertClickTest < ActiveSupport::TestCase assert_equal 1, advert.reload.num_clicks end - - test "doesn't log for admin users" do - advert = create :advert - user = create :user, :admin - assert_equal 0, advert.num_clicks - - Partner::LogAdvertClick.(advert, user, nil, nil) - - assert_equal 0, advert.reload.num_clicks - end end