Skip to content

chore: updated value for socialAI#8536

Merged
dovydas55 merged 2 commits intomainfrom
chore/update-traderProfileIds-to-mutedTraderProfileIds
Apr 21, 2026
Merged

chore: updated value for socialAI#8536
dovydas55 merged 2 commits intomainfrom
chore/update-traderProfileIds-to-mutedTraderProfileIds

Conversation

@zone-live
Copy link
Copy Markdown
Contributor

@zone-live zone-live commented Apr 21, 2026

Explanation

Updates the traderProfileIds to mutedTraderProfileIds. So that its in line with the latest spec.

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Medium Risk
Breaking rename in a public type/validator can cause downstream compile-time failures and runtime payload mismatches until consumers update.

Overview
Renames the SocialAI notification preference field from traderProfileIds to mutedTraderProfileIds across exported types, superstruct validation, and test mocks to align with the API payload.

Updates the package changelog to flag this as a BREAKING change for consumers relying on the old field name.

Reviewed by Cursor Bugbot for commit e6dc891. Bugbot is set up for automated code reviews on this repo. Configure here.

@zone-live zone-live requested a review from a team as a code owner April 21, 2026 08:46
@zone-live zone-live requested a review from a team as a code owner April 21, 2026 08:50
@dovydas55 dovydas55 added this pull request to the merge queue Apr 21, 2026
Merged via the queue into main with commit a0a0972 Apr 21, 2026
358 checks passed
@dovydas55 dovydas55 deleted the chore/update-traderProfileIds-to-mutedTraderProfileIds branch April 21, 2026 09:12
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