Add analytic query param to videos#find_by_id

Whenever the user comes to a video page by using the shortlink URL (/v/1234), there will be an extra analytic parameter as ?ref=shortlink so that I can see in Plausible the usage of the shortlink feature to analyze the performance of the links that I usually place at the video description in YouTube.
makigas · Aug 26, 2024 · 6f670da · 6f670da
1 parent 68c62eb
commit 6f670da
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/controllers/videos_controller.rb b/app/controllers/videos_controller.rb
@@ -25,7 +25,7 @@ def show
 
   def find_by_id
     @video = Video.find_by!(youtube_id: params[:id])
-    redirect_to playlist_video_path(@video, playlist_id: @video.playlist)
+    redirect_to playlist_video_path(@video, playlist_id: @video.playlist, ref: 'shortlink')
   end
 
   def early

diff --git a/spec/features/videos/video_find_by_id_spec.rb b/spec/features/videos/video_find_by_id_spec.rb
@@ -9,7 +9,7 @@
   describe 'when the correct video is found' do
     it 'redirects to the correct video page' do
       visit "/v/#{video.youtube_id}"
-      expect(page).to have_current_path playlist_video_path(video, playlist_id: playlist)
+      expect(page).to have_current_path playlist_video_path(video, playlist_id: playlist, ref: 'shortlink')
     end
   end