Skip to content

Add optional aliasids filter to player-relationships broadcast - #1092

Merged
tudddorrr merged 1 commit into
developfrom
broadcast-to-list
Oct 13, 2026
Merged

tudddorrr merged 1 commit into
developfrom
broadcast-to-list

Conversation

@tudddorrr

Copy link
Copy Markdown
Collaborator

Broadcast targeting

  • Added an optional aliasIds filter to the v1.player-relationships.broadcast socket message so callers can limit a broadcast to specific confirmed subscribers.
  • When aliasIds is omitted or empty, behavior is unchanged and the message goes to every confirmed subscriber.
  • Alias IDs not tied to a confirmed subscription are ignored.

Validation

  • aliasIds must be an array of positive integer IDs and is capped at 1000 entries, with invalid input rejected as INVALID_MESSAGE_DATA.

@tudddorrr tudddorrr added the enhancement New feature or request label Oct 13, 2026
@codecov

codecov Bot commented Oct 13, 2026

Copy link
Copy Markdown

Codecov Report

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

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1092      +/-   ##
===========================================
+ Coverage    98.73%   98.75%   +0.02%     
===========================================
  Files          530      532       +2     
  Lines         8153     8210      +57     
  Branches      1059     1068       +9     
===========================================
+ Hits          8050     8108      +58     
  Misses          36       36              
+ Partials        67       66       -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 merged commit 1b51548 into develop Oct 13, 2026
9 checks passed
@tudddorrr
tudddorrr deleted the broadcast-to-list branch October 13, 2026 07:41
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