Skip to content

chore(analytics): remove dead Amplitude events and LISTEN_GATED#14304

Open
dylanjeffers wants to merge 3 commits into
mainfrom
chore/remove-dead-amplitude-events
Open

chore(analytics): remove dead Amplitude events and LISTEN_GATED#14304
dylanjeffers wants to merge 3 commits into
mainfrom
chore/remove-dead-amplitude-events

Conversation

@dylanjeffers
Copy link
Copy Markdown
Contributor

Summary

  • Removes zero-callsite Name enum entries: COMMENTS_REPLY_TO_COMMENT, CREATOR_NODE_SELECTION, SEARCH_TAB_CLICK, SERVICE_MONITOR_HEALTH_CHECK, SERVICE_MONITOR_REQUEST, SIGN_UP_REPAIR_START/SUCCESS/FAILURE
  • Removes deprecated social OAuth signup event families: all CREATE_ACCOUNT_*_TWITTER, _INSTAGRAM, _TIKTOK variants (enum entries, payload types, union members)
  • Removes BUY_AUDIO family events (BUY_AUDIO_SUCCESS, BUY_AUDIO_FAILURE, BUY_AUDIO_ON_RAMP_*, BUY_AUDIO_RECOVERY_*) replaced by BUY_USDC equivalents
  • Removes LISTEN_GATED fully: enum entry, ListenGated payload type, union member, and callsite in recordListen.ts
  • Removes ARTIST_COIN_PURCHASE and ARTIST_RECOMMENDATIONS_POPUP from FollowSource enum and updates their callsites to FollowSource.OVERFLOW / FollowSource.USER_LIST

Test plan

  • `npm run typecheck` passes in `packages/common` and `packages/web`
  • No references to removed enum values remain in the codebase

🤖 Generated with Claude Code

dylanjeffers and others added 3 commits May 8, 2026 14:42
Instruments three key contest interactions following the existing
'Remix Contest:' event prefix and {remixContestId, trackId} property
shape used by the host/pick-winners events:

  - REMIX_CONTEST_VIEW: contest page/screen first resolves trackId+eventId
  - REMIX_CONTEST_ENTER: user taps Enter Contest / Upload Remix
  - REMIX_CONTEST_VIEW_SUBMISSIONS: user opens the submissions tab

Mobile submissions-tab firing uses useFocusedTab from
react-native-collapsible-tab-view because the contest tabs mount
eagerly (lazy: false) — a plain mount effect would fire even for
users who only view the Details tab.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…t sections

Bug 1 (web): Align static explore section visibility with SearchResults render
condition — hide sections when inputValue is set (during debounce window) in
addition to when showSearchResults is true, so Premium and other filters applied
via the filter pills always suppress unfiltered curated sections.

Bug 2 (web): Guard empty-state tile behind !isPending in AlbumResultsPage and
PlaylistResultsPage so the NoResultsTile is not shown while the query is still
in its initial loading phase, preventing a spurious empty-state flash when
combining "Downloads Available" + "Electronic" genre filters.

Bug 3 (mobile): Include category !== 'all' in the showSearch guard on the
explore screen so tapping a content-type pill (Tracks, Albums, etc.) immediately
shows the filtered SearchResults without requiring a second filter to be added.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Removes zero-callsite Name enum entries (COMMENTS_REPLY_TO_COMMENT,
CREATOR_NODE_SELECTION, SEARCH_TAB_CLICK, SERVICE_MONITOR_HEALTH_CHECK,
SERVICE_MONITOR_REQUEST, SIGN_UP_REPAIR_*), deprecated social OAuth signup
events (CREATE_ACCOUNT_*_TWITTER/INSTAGRAM/TIKTOK), BUY_AUDIO family events
replaced by BUY_USDC, LISTEN_GATED with its callsite and payload type,
and ARTIST_COIN_PURCHASE/ARTIST_RECOMMENDATIONS_POPUP from FollowSource.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 13, 2026

⚠️ No Changeset found

Latest commit: d6db033

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant