Skip to content

Conversation

@samholmes
Copy link
Contributor

@samholmes samholmes commented Oct 31, 2025

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Requirements

If you have made any visual changes to the GUI. Make sure you have:

  • Tested on iOS device
  • Tested on Android device
  • Tested on small-screen device (iPod Touch)
  • Tested on large-screen device (tablet)

Note

Fixes broken analytics by dispatching logEvent in Redux and plumbing an onLogEvent callback through fiat plugins, with PostHog global typing and usage updates.

  • Analytics / Tracking:
    • Replace direct logEvent(...) with dispatch(logEvent(...)) in AccountReferralActions, RampCreateScene, and WalletListCreateRow.
    • Add onLogEvent callback to fiat plugin APIs (fiatPlugin.tsx, fiatPluginTypes.ts) and use it in amountQuotePlugin for Buy_Quote / Sell_Quote and *_Next events.
    • Add global PostHog typings (src/types/posthog.d.ts) and update tracking.ts to set/use global.posthog (remove TypeScript ignores).
  • Type/Code Hygiene:
    • Add explicit React return types and stricter error typings in WalletListCreateRow.
  • Docs:
    • Update CHANGELOG.md to note tracking fix and removal of change quote tracking.

Written by Cursor Bugbot for commit 957d4ce. This will update automatically on new commits. Configure here.


@samholmes samholmes force-pushed the sam/posthog-capture-event-fixes branch from e9ae97a to 957d4ce Compare November 3, 2025 22:05
@samholmes samholmes enabled auto-merge November 3, 2025 22:05
@samholmes samholmes merged commit 39363b3 into develop Nov 3, 2025
3 checks passed
@samholmes samholmes deleted the sam/posthog-capture-event-fixes branch November 3, 2025 22:29
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.

3 participants