Skip to content

Add POST /player/basic/batch - #147

Merged
owens1127 merged 3 commits into
mainfrom
fix-player-basic-batch
Jun 17, 2026
Merged

Add POST /player/basic/batch#147
owens1127 merged 3 commits into
mainfrom
fix-player-basic-batch

Conversation

@owens1127

Copy link
Copy Markdown
Contributor

Summary

  • New POST /player/basic/batch accepts up to 12 membershipIds and returns found players in one SQL round-trip (WHERE membership_id = ANY(...)).
  • Companion to Website prevention infra PR — eliminates fireteam N+1 /player/*/basic calls.

Test plan

  • bun test src/routes/player/basic-batch.test.ts (needs local Postgres fixture DB)
  • Merge before Website PR that consumes this endpoint
  • After deploy: curl -X POST https://api.raidhub.io/player/basic/batch -H "X-API-KEY: ..." -d '{"membershipIds":["..."]}'

Made with Cursor

owens1127 and others added 2 commits June 16, 2026 22:34
Single round-trip for up to 12 membership IDs using WHERE membership_id = ANY(...). Used by Website fireteam display to avoid N+1 basic calls.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@barecheck

barecheck Bot commented Jun 17, 2026

Copy link
Copy Markdown

Barecheck - Code coverage report

Total: 91.5%

Your code coverage diff: 0.03% ▴

Uncovered files and lines
FileLines
src/services/player.ts32

@owens1127
owens1127 merged commit 20fd56e into main Jun 17, 2026
7 checks passed
@owens1127
owens1127 deleted the fix-player-basic-batch branch June 17, 2026 02:49
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.

1 participant