Skip to content

fix(api): implement pagination and bounded concurrency for broadcast …#1907

Merged
dipexplorer merged 4 commits into
RatLoopz:mainfrom
VaishnaviL2005:fix/broadcast-pagination-bug
Jun 15, 2026
Merged

fix(api): implement pagination and bounded concurrency for broadcast …#1907
dipexplorer merged 4 commits into
RatLoopz:mainfrom
VaishnaviL2005:fix/broadcast-pagination-bug

Conversation

@VaishnaviL2005

Copy link
Copy Markdown
Contributor

🛑 STOP: Assignment & File Scope Check

  • I am assigned to this issue.
  • I verified that this PR ONLY touches the required files.

📋 PR Summary & Link

  • **Closes [BUG] Notification broadcast route truncates recipients at Supabase max_rows limit #1905 **
  • Implemented Offset Pagination: Fixes broadcast recipient truncation caused by Supabase max_rows limits by introducing paginated subscriber retrieval using .range() and explicit .order("id").
  • Added Bounded-Concurrency: Additionally improves delivery throughput by processing recipients in chunks of 50, respecting external SMS/WhatsApp provider rate limits without sequential blocking.

📸 Proof of Work (Screenshots / Logs)

image image

🏷️ PR Type

  • 🐛 type: bug
  • type: feature
  • 📖 type: docs
  • 🧪 type: testing
  • 🔒 type: security
  • type: performance
  • 🎨 type: design
  • ♻️ type: refactor
  • 🛠️ type: devops
  • type: accessibility

✅ Checklist

  • My PR has a linked issue (Closes #1905 )
  • I have pulled the latest main and resolved any conflicts

@github-project-automation github-project-automation Bot moved this to 📥 Backlog in SahiDawa Workflow Jun 14, 2026
@github-actions github-actions Bot added gssoc:approved Approved for gssoc type:bug Something isn't working labels Jun 14, 2026
@dipexplorer dipexplorer added level:advanced 55 pts quality:clean multiplier x1.2 type:refactor Code structure cleanup without behavior change type:performance Performance optimization or latency improvements labels Jun 15, 2026
@dipexplorer dipexplorer merged commit 1e6dd4c into RatLoopz:main Jun 15, 2026
24 of 27 checks passed
@github-project-automation github-project-automation Bot moved this from 📥 Backlog to 🎉 Merged in SahiDawa Workflow Jun 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🎉 Congratulations @VaishnaviL2005! Your Pull Request "fix(api): implement pagination and bounded concurrency for broadcast …" has been successfully merged by @dipexplorer.

Thank you for your valuable contribution to SahiDawa! 🇮🇳
If this was for GSSoC 2026, your work is officially merged and valid. Keep up the great work and feel free to claim other open issues. 🚀

Follow us on LinkedIn: https://www.linkedin.com/company/ratloopz/ to get shoutout

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:approved Approved for gssoc level:advanced 55 pts quality:clean multiplier x1.2 type:bug Something isn't working type:performance Performance optimization or latency improvements type:refactor Code structure cleanup without behavior change

Projects

Status: 🎉 Merged

Development

Successfully merging this pull request may close these issues.

[BUG] Notification broadcast route truncates recipients at Supabase max_rows limit

2 participants