Skip to content

Add player auth activity enrichment setting per game - #1066

Merged
tudddorrr merged 1 commit into
developfrom
player-auth-activity-enrichment-setting
Sep 13, 2026
Merged

tudddorrr merged 1 commit into
developfrom
player-auth-activity-enrichment-setting

Conversation

@tudddorrr

Copy link
Copy Markdown
Collaborator

Enrichment setting

  • New playerAuthActivityEnrichment boolean on the Game entity, defaulting to false
  • Added via migration AddPlayerAuthActivityEnrichmentColumn

Activity logging

  • buildPlayerAuthActivity now includes userAgent and ip in the activity extra only when the game has opted in
  • Merge route updated to use buildPlayerAuthActivity and populate the player's game relation after fork

Dashboard settings

  • Protected game settings endpoint returns playerAuthActivityEnrichment
  • Game update endpoint accepts and persists the new playerAuthActivityEnrichment field

@codecov

codecov Bot commented Sep 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.48%. Comparing base (2bc97a2) to head (40b3fe1).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1066      +/-   ##
===========================================
- Coverage    98.49%   98.48%   -0.02%     
===========================================
  Files          488      488              
  Lines         7837     7840       +3     
  Branches      1027     1029       +2     
===========================================
+ Hits          7719     7721       +2     
  Misses          49       49              
- Partials        69       70       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tudddorrr
tudddorrr force-pushed the player-auth-activity-enrichment-setting branch from f8cdd28 to 2c38ce7 Compare September 13, 2026 22:32
@tudddorrr
tudddorrr force-pushed the player-auth-activity-enrichment-setting branch from 2c38ce7 to 40b3fe1 Compare September 13, 2026 22:38
@tudddorrr tudddorrr added the enhancement New feature or request label Sep 13, 2026
@tudddorrr
tudddorrr enabled auto-merge September 13, 2026 22:40
@tudddorrr
tudddorrr merged commit 4f5c1b3 into develop Sep 13, 2026
7 checks passed
@tudddorrr
tudddorrr deleted the player-auth-activity-enrichment-setting branch September 13, 2026 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant