Skip to content

Conversation

@cmonfortep
Copy link
Contributor

@cmonfortep cmonfortep commented Oct 29, 2025

Task/Issue URL: https://app.asana.com/1/137249556945/project/72649045549333/task/1211382557175161?focus=true

Description

Implements attributed metrics around Duck ai usage

Steps to test this PR

Use logcat filter tag~:"AttributedMetrics"

Feature 1

  • Fresh install (including removing ddg folder)
  • Ensure Client status running: true -> isActive: true, isEnabled: true
  • skip onboarding
  • Go to attributed metrics and move the install date back in time
  • Open Duck.ai
  • Send a message
  • Ensure Collected event submit_prompt
  • Also AdClickCount7d: Skip emitting, not enough data or no events
  • Go to attributed metrics and Add few DuckAI fake events (click on the option)
  • Open Duck.ai
  • Send a message
  • Ensure Fired pixel user_average_duck_ai_usage_past_week with params

Feature 2

  • With aiUsageAvg disabled we don't collect and we don't emit
  • Interact with duck ai and confirm no collected events, like feature 100% disabled
  • With aiUsageAvg enabled, but canEmitAIUsageAvg disabled, we collect but don't emit
  • Interact with duck ai and confirm we collect events (check logs), but we don't emit

UI changes

Before After
!(Upload before screenshot) (Upload after screenshot)

@cmonfortep
Copy link
Contributor Author

cmonfortep commented Oct 29, 2025

@cmonfortep cmonfortep force-pushed the feature/cristian/duck_ai_attributed_metric branch from e310bbc to 283496f Compare October 29, 2025 18:29
@cmonfortep cmonfortep marked this pull request as ready for review October 29, 2025 18:29
@cmonfortep cmonfortep force-pushed the feature/cristian/duck_ai_attributed_metric branch from 283496f to d3ce0d5 Compare October 29, 2025 20:34
@cmonfortep cmonfortep force-pushed the feature/cristian/attributed_remote_config branch from 85c065a to 9d6f764 Compare October 29, 2025 21:23
@cmonfortep cmonfortep force-pushed the feature/cristian/duck_ai_attributed_metric branch from d3ce0d5 to 7a065ec Compare October 29, 2025 21:24
@cmonfortep cmonfortep force-pushed the feature/cristian/attributed_remote_config branch from 9d6f764 to 835a3c3 Compare October 29, 2025 22:32
@cmonfortep cmonfortep force-pushed the feature/cristian/duck_ai_attributed_metric branch from 7a065ec to b571b15 Compare October 29, 2025 22:32
@cmonfortep cmonfortep force-pushed the feature/cristian/attributed_remote_config branch from 835a3c3 to fca3fe3 Compare October 29, 2025 23:10
@cmonfortep cmonfortep force-pushed the feature/cristian/duck_ai_attributed_metric branch from b571b15 to cafc989 Compare October 29, 2025 23:11
@cmonfortep cmonfortep force-pushed the feature/cristian/attributed_remote_config branch from fca3fe3 to 7d68495 Compare October 30, 2025 09:01
@cmonfortep cmonfortep force-pushed the feature/cristian/duck_ai_attributed_metric branch from cafc989 to 4316a72 Compare October 30, 2025 09:01
Copy link
Contributor

@marcosholgado marcosholgado left a comment

Choose a reason for hiding this comment

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

LGTM

@cmonfortep cmonfortep force-pushed the feature/cristian/attributed_remote_config branch from 7d68495 to 27f3c83 Compare October 31, 2025 11:56
@cmonfortep cmonfortep force-pushed the feature/cristian/duck_ai_attributed_metric branch from 4316a72 to 893affe Compare October 31, 2025 11:56
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