Skip to content

Add analytics event for outcomes, if analytics enabled#290

Merged
dektar merged 2 commits intomasterfrom
outcomes
Feb 16, 2026
Merged

Add analytics event for outcomes, if analytics enabled#290
dektar merged 2 commits intomasterfrom
outcomes

Conversation

@dektar
Copy link
Collaborator

@dektar dektar commented Feb 16, 2026

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization

Description

Adds plausible calls for outcomes

Were the changes tested?

  • Yes, automated tests in please name test methods or files
  • Yes, manually tested: Commented out the if check for analytics and logged an unavailable, checked plausible for it
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

@dektar dektar requested a review from nickoneill February 16, 2026 02:00
Copy link
Member

@nickoneill nickoneill left a comment

Choose a reason for hiding this comment

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

lgtm!

@dektar dektar merged commit ad6d41a into master Feb 16, 2026
1 check passed
@dektar dektar deleted the outcomes branch February 16, 2026 23:03
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.

2 participants