From 70c80c2ba9336407c413560390c3e47ba8ca8917 Mon Sep 17 00:00:00 2001
From: Matthew Landauer <mlandauer@gmail.com>
Date: Fri, 22 Nov 2024 00:19:33 +0000
Subject: [PATCH] Make share button component automagically add utm tags

---
 app/components/share_button_component.html.erb      |  4 ++--
 app/components/share_button_component.rb            | 13 +++++++++++++
 .../applications/_thanks_for_commenting.html.erb    | 10 +++-------
 app/views/applications/show.html.erb                | 10 +++-------
 4 files changed, 21 insertions(+), 16 deletions(-)

diff --git a/app/components/share_button_component.html.erb b/app/components/share_button_component.html.erb
index ca68e3c2c..3e8e30530 100644
--- a/app/components/share_button_component.html.erb
+++ b/app/components/share_button_component.html.erb
@@ -7,7 +7,7 @@
   x-bind:class="{'invisible': false}"
   x-data="{
     copied: false,
-    shareData: {url: '<%= j(@url) %>', title: '<%= j(@title) %>'}
+    shareData: {url: '<%= j(url_with_tracking) %>', title: '<%= j(@title) %>'}
   }">
   <button class="inline-block px-10 py-3 text-xl font-semibold border-2 sm:py-4"
           x-bind:class="{'bg-white': !copied, 'bg-light-grey2': copied,
@@ -21,7 +21,7 @@
               console.log('Web share error', err);
             }
           } else {
-            navigator.clipboard.writeText('<%= @url %>');
+            navigator.clipboard.writeText('<%= url_with_tracking %>');
             copied = true;
             setTimeout(() => { copied = false }, 2000);
           }">
diff --git a/app/components/share_button_component.rb b/app/components/share_button_component.rb
index 19691d542..c8e6986bd 100644
--- a/app/components/share_button_component.rb
+++ b/app/components/share_button_component.rb
@@ -19,4 +19,17 @@ def initialize(url:, title:, color:)
       raise "Unexpected color: #{color}"
     end
   end
+
+  sig { returns(String) }
+  def url_with_tracking
+    append_params(@url, utm_source: "share", utm_content: content.strip)
+  end
+
+  sig { params(url: String, params: T::Hash[String, String]).returns(String) }
+  def append_params(url, params)
+    u = URI.parse(url)
+    e = Rack::Utils.parse_nested_query(u.query).merge(params).to_param
+    u.query = e
+    u.to_s
+  end
 end
diff --git a/app/views/applications/_thanks_for_commenting.html.erb b/app/views/applications/_thanks_for_commenting.html.erb
index a6b5e1980..3d3f541c9 100644
--- a/app/views/applications/_thanks_for_commenting.html.erb
+++ b/app/views/applications/_thanks_for_commenting.html.erb
@@ -33,13 +33,9 @@
       <p class="mb-8">
         Thanks, your comment has been sent to <%= comment.comment_recipient_full_name %> and posted below!
       </p>
-      <%# TODO: Make the share button component add the utm tags automatically %>
-      <%= c = "Share your comment"
-          render ShareButtonComponent.new(
-            url: comment_url(comment, utm_source: "share", utm_content: c),
-            title: "My comment on #{page_title(comment.application)}",
-            color: :lavender
-          ).with_content(c) %>
+      <%= render ShareButtonComponent.new(url: comment_url(comment), title: "My comment on #{page_title(comment.application)}", color: :lavender) do %>
+        Share your comment
+      <% end %>
     </div>
   <% end %>
   <%# Make the canvas (which is used to show the confetti) the same size as the alert %>
diff --git a/app/views/applications/show.html.erb b/app/views/applications/show.html.erb
index fc2ab98f6..7b6488ad6 100644
--- a/app/views/applications/show.html.erb
+++ b/app/views/applications/show.html.erb
@@ -97,13 +97,9 @@
 </div>
 
 <div class="mt-6 md:mt-2">
-  <%# TODO: Make the share button component add the utm tags automatically %>
-  <%= c = "Share this application"
-      render ShareButtonComponent.new(
-        url: application_url(@application, utm_source: "share", utm_content: c),
-        title: page_title(@application),
-        color: :green
-      ).with_content(c) %>
+  <%= render ShareButtonComponent.new(url: application_url(@application), title: page_title(@application), color: :green) do %>
+    Share this application
+  <% end %>
 </div>
 
 <%= render "applications/create_alert_form_bigger", alert: @alert %>