Expand ahoy #835

maebeale · 2026-02-02T17:14:24Z

This work Closes [link an issue]

What is the goal of this PR and why is this important?

Add more events to ahoy activity tracking. Rearrange admin and activity-related files. Surface ahoy events on a new charts page.

- Model lifecycle Events:
    - create.<resource_name>
    - update.<resource_name>
    - destroy.<resource_name>
- Controller Events (from AhoyTracking):  
    - browse.taggings
    - download.resource
    - view.<resource_name>
    - print.<resource_name>
- Auth Events on User:  
    - auth.account_confirmed (triggered after devise confirmation)
    - auth.account_locked (triggered after devise account lock)
    - auth.account_unlocked (triggered after devise account unlock)
    - auth.email_changed (triggered when the email is changed)
    - auth.account_deleted (triggered before account deletion)

Move recent_activities out of dashboard and into Admin::AhoyActivitiesController
Update AhoyViewTracking -> AhoyTracking, since it's tracking more than just show page events now
Change activities/counts to cluster prints and downloads within the model square
Added model concern to have automatic callbacks to track_lifecycle_event without littering all controllers
- This is separate from the track_event and track_index
Added callbacks to User for tracking auth events
Added method to track browse.taggings and also to capture search queries on Workshops and Resources
Added new /activities/charts endpoint that shows various charts based on ahoy data
- This required adding chartkick, groupdate, geocoder, and maxmind-geoip2

How did you approach the change?

Added EventBuilder to construct/assign extra fields to ahoy events
Added ActivityPresenter to convert an ahoy Event to its corresponding record
Added current.rb to make user accessible to models (so it can be added to auth ahoy events)
Added already_tracked? to prevent duplicate activities from being created as Events within the same Visit
Added LifecycleBuffer to push and flush data from the store (to separate model-related ahoy data from models)
Added admin? method on user so we can move away from user.super_user designation later

Anything else to add?

Move /admin out of dashboard and into Admin::HomeController
Added a Rails.logger for when a notification is created
Bulk changed a bunch of rubocop violations

jmilljr24

Good stuff!

I see there is a start with the action_policy stuff. I can finish it off in another PR if its too much to include in this one.

maebeale · 2026-02-03T01:22:46Z

CONTRIBUTING.md

 - To see your data
  - The home page will show Workshops, CommunityNews, Resources, Events, and Stories
-  - The [Admin Dashboard](http://localhost:3000/dashboard/admin) provides CRUD access for most models
+  - The [Admin Home](http://localhost:3000/admin) provides CRUD access for most models


"Admin Dashboard" wasn't accurate since it doesn't have any analytics on it. And also since we use the word Dashboard a lot wrt the old site, it seemed smart to change this to "Admin Home" instead.

maebeale · 2026-02-03T01:23:35Z

spec/requests/admin/analytics_spec.rb


      it "denies access to analytics page" do
-        get "/admin/analytics"
+        get "/admin/activities/counts"


analytics was a little too vague. this is really just rolling up ahoy event (user activity) counts.

jmilljr24 · 2026-02-03T01:33:27Z

Gemfile

  gem "dotenv-rails"
  gem "faker"
  gem "factory_bot_rails"
+  gem "launchy"


Suggested change

gem "launchy"

This is in group :test already.

maebeale · 2026-02-03T15:21:49Z

db/migrate/20260131193429_add_resource_dimensions_to_ahoy_events.rb

@@ -0,0 +1,9 @@
+class AddResourceDimensionsToAhoyEvents < ActiveRecord::Migration[8.1]
+  def change
+    add_column :ahoy_events, :resource_type, :string


ahoy by default stuffs everything into properties, but since we're querying and running charts, i wanted to pull some data into columns

maebeale · 2026-02-03T15:22:21Z

config/routes.rb

  get "contact_us", to: "contact_us#index"
  post "contact_us", to: "contact_us#create"
-  get "dashboard/admin", to: "dashboard#admin"
-  get "dashboard/recent_activities", to: "dashboard#recent_activities"


these didn't belong in DashboardController. that should only run the site home page.

maebeale · 2026-02-03T15:23:46Z

config/routes.rb

+    get "/",                  to: "home#index"
+    get "activities",          to: "ahoy_activities#index", as: "activities"
+    get "activities/charts",   to: "ahoy_activities#charts", as: "activities_charts"
+    get "activities/recent",   to: "ahoy_activities#recent", as: "activities_recent"


still on the fence about /activities/recent since that content is now at /activities/. still might be good to have a separate, faster page that's just pulling most recent 50 activities.

maebeale · 2026-02-03T15:24:33Z

config/initializers/geocoder.rb

@@ -0,0 +1,6 @@
+Geocoder.configure(


this is needed for the charts, incl the "Top Cities". maxmind requires you to store this mmdb file locally, so that's why it's referring to that file on line 4.

maebeale · 2026-02-03T15:25:00Z

config/initializers/ahoy.rb

+# customize Ahoy::Event to extract resource dimensions from properties
+Rails.application.config.to_prepare do
+  Ahoy::Event.class_eval do
+    before_validation :extract_resource_dimensions, on: :create


this callback is needed in order to populate those extra fields on ahoy events table

maebeale · 2026-02-03T15:25:48Z

app/views/taggings/index.html.erb


        <%= render "tagged_items_carousel", items: items %>
+
+        <div class="mt-6 text-right">


ux update while i was there. added a linked text in bottom right so it functions just like the site landing page.

maebeale · 2026-02-03T15:27:12Z

app/views/shared/_navbar_user.html.erb


  </button>
  <!-- Dropdown menu -->
-<div


this is rearranging user dropdown. this must be coming from rebase.

maebeale · 2026-02-03T15:27:39Z

app/views/shared/_form_field.html.erb

@@ -15,30 +15,30 @@
  <div class="row">


all changes in this file are rubocop

maebeale · 2026-02-03T15:27:51Z

app/views/reports/share_story.html.erb

@@ -19,7 +19,7 @@
             <%= f.hidden_field :type %>


rubocop changes in this file

maebeale · 2026-02-03T15:28:53Z

app/views/categories/_form.html.erb

      <div class="flex-1 min-w-[220px]">
        <%= f.input :category_type_id,
                    label: "Category Type",
+                    required: true,


this is required by validation but was missing from form

maebeale · 2026-02-03T15:30:12Z

app/services/notification_services/create_notification.rb

      )
-
-
+      Rails.logger.info({


decided not to create ahoy events for notifications. so, adding to logs for visibility.

maebeale · 2026-02-03T15:30:29Z

app/services/analytics/lifecycle_buffer.rb

@@ -0,0 +1,22 @@
+# app/services/analytics/lifecycle_buffer.rb


this is needed so we don't have to pollute models w ahoy logic

maebeale · 2026-02-03T15:31:10Z

app/services/analytics/event_builder.rb

+# app/services/analytics/event_builder.rb
+module Analytics
+  class EventBuilder
+    def self.lifecycle(action, resource, user: nil)


this is needed so we can append the custom data to our custom ahoy fields (resource_type, resource_id, resource_title)

maebeale · 2026-02-03T15:31:30Z

app/services/analytics/ahoy_tracker.rb

+      end
+
+      def safe_for_tracking?(attribute)
+        !attribute.match?(/password|token|secret|key|digest|salt|otp/i)


don't store these fields in ahoy

…'s being tracked

…Add ahoy charts that required geo gems.

…cludes/joins bc joins doesn't preload.

…or AS records. only capture after_update if true update and not a create.

maebeale · 2026-02-03T17:03:38Z

yes please, @jmilljr24 !

maebeale force-pushed the expand-ahoy branch from a1fe975 to 48e911d Compare February 2, 2026 17:16

jmilljr24 approved these changes Feb 3, 2026

View reviewed changes

maebeale commented Feb 3, 2026

View reviewed changes

jmilljr24 reviewed Feb 3, 2026

View reviewed changes

maebeale commented Feb 3, 2026

View reviewed changes

app/views/shared/_form_field.html.erb

@@ -15,30 +15,30 @@

<div class="row">

Copy link

Collaborator Author

maebeale Feb 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all changes in this file are rubocop

maebeale commented Feb 3, 2026

View reviewed changes

app/views/reports/share_story.html.erb

@@ -19,7 +19,7 @@

<%= f.hidden_field :type %>

Copy link

Collaborator Author

maebeale Feb 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rubocop changes in this file

maebeale commented Feb 3, 2026

View reviewed changes

maebeale added 7 commits February 3, 2026 11:59

Rename AhoyViewTracking to AhoyTracking bc it's not only viewing that…

91b8a45

…'s being tracked

Move recent_activity under admin/ahoy_activities. Update ahoy usage. …

79af71c

…Add ahoy charts that required geo gems.

Expand ahoy

1e63344

Move dashboard/admin under /admin

9f4ad0f

Update ahoy to work as callback on all model lifecycle events. fix in…

fb2b0c8

…cludes/joins bc joins doesn't preload.

Log notification created. Don't create ahoy events for Notifications …

9a729c8

…or AS records. only capture after_update if true update and not a create.

Remove events code and add charts to admin page

e3f6dbc

maebeale force-pushed the expand-ahoy branch from 327ecb0 to e3f6dbc Compare February 3, 2026 17:00

maebeale merged commit 783eec4 into main Feb 3, 2026
3 checks passed

maebeale deleted the expand-ahoy branch February 3, 2026 17:05


		<%= render "tagged_items_carousel", items: items %>

		<div class="mt-6 text-right">

		@@ -0,0 +1,22 @@
		# app/services/analytics/lifecycle_buffer.rb

Expand ahoy #835

Expand ahoy #835

Conversation

maebeale commented Feb 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What is the goal of this PR and why is this important?

How did you approach the change?

Anything else to add?

Uh oh!

jmilljr24 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

maebeale commented Feb 3, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

maebeale commented Feb 2, 2026 •

edited

Loading