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/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; } 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