Skip to content

Record subscription, mailing list, and profile activity - #2851

Merged
mroderick merged 2 commits into
masterfrom
feat/activity-subscriptions-profile
Sep 13, 2026
Merged

Record subscription, mailing list, and profile activity#2851
mroderick merged 2 commits into
masterfrom
feat/activity-subscriptions-profile

Conversation

@mroderick

@mroderick mroderick commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

Second of the activity-log stack (base: #2850). Adds member subscription and profile activity: subscription.created / subscription.removed (group subscriptions), mailing_list.subscribe / mailing_list.unsubscribe, profile.updated (both profile forms), and toc.accepted.

Key changes

  • Instrumented at controller level where the actor is known; no model callbacks.
  • Mailing-list keys are distinct from chapter group subscriptions.

Review notes

Recording happens at the point of intent (e.g. before the newsletter service call) — flagged in review as a fidelity nuance for external calls; left as-is for this stack.

Stack: PR 3 adds RSVP/waiting-list/check-in activity.


Context and motivation: #2857

@mroderick
mroderick force-pushed the feat/activity-subscriptions-profile branch from 7f25338 to c9f8260 Compare September 9, 2026 09:56
Base automatically changed from feat/activity-recorder to master September 9, 2026 10:11
@mroderick
mroderick force-pushed the feat/activity-subscriptions-profile branch from c9f8260 to 1aff4dc Compare September 9, 2026 10:11
@mroderick
mroderick marked this pull request as ready for review September 9, 2026 10:38
@mroderick
mroderick force-pushed the feat/activity-subscriptions-profile branch from 1aff4dc to ed38bba Compare September 11, 2026 11:00
@mroderick
mroderick force-pushed the feat/activity-subscriptions-profile branch from ed38bba to 8f2d284 Compare September 13, 2026 10:10
@mroderick
mroderick merged commit 18da65d into master Sep 13, 2026
18 of 19 checks passed
@mroderick
mroderick deleted the feat/activity-subscriptions-profile branch September 13, 2026 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant