Skip to content

Endpoint for batch rule subscription#517

Merged
bracyw merged 2 commits intodevelopfrom
497-endpoint-for-batch-rule-subscription
Feb 17, 2026
Merged

Endpoint for batch rule subscription#517
bracyw merged 2 commits intodevelopfrom
497-endpoint-for-batch-rule-subscription

Conversation

@bracyw
Copy link
Collaborator

@bracyw bracyw commented Feb 16, 2026

Changes

Added POST /rules/subscribe endpoint for batch rule subscription. Takes rule_ids and client_id in request body.

Implementation validates all rules exist before subscribing to any (atomic behavior). O(n) time complexity as specified.

Test Cases

  • Success case with multiple rules
  • Error handling for nonexistent rules
  • Empty list edge case
  • Concurrent subscription stress test

All tests pass.

Checklist

  • All commits are tagged with the ticket number
  • No linting errors / newline at end of file warnings
  • All code follows repository-configured prettier formatting
  • No merge conflicts
  • All checks passing
  • Remove any non-applicable sections of this template
  • Assign the PR to yourself
  • Request reviewers & ping on Slack
  • PR is linked to the ticket (see below)

Closes #497

@bracyw bracyw changed the base branch from main to develop February 16, 2026 21:33
@bracyw bracyw requested a review from suryatho February 16, 2026 21:45
@bracyw bracyw self-assigned this Feb 17, 2026
@bracyw bracyw merged commit 5a7eacc into develop Feb 17, 2026
4 checks passed
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.

Endpoint for batch rule subscription

2 participants