Implement CSV export endpoint with streaming support - #937
Conversation
- Updated GET /api/streams/export.csv to match acceptance criteria - Added all required columns: id, sender, recipient, asset, totalAmount, vestedAmount, claimedAmount, status, startAt, durationSeconds, createdAt - Implemented all filters from list endpoint: status, sender, recipient, asset, assetCode, q, minAmount, maxAmount, include_archived - Added streaming support for large exports using res.write() instead of building entire CSV in memory - Proper CSV escaping for Excel/Sheets compatibility (handles commas, quotes, newlines) - Updated Swagger documentation with complete parameter list and response schema - Updated integration tests to verify new columns and additional filters - Removed caching for export endpoint to ensure fresh data on large exports Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
|
@dumbdevss is attempting to deploy a commit to the ritik4ever's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@dumbdevss Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Generated with Devin
Commit Type
Breaking change? (If yes, add
BREAKING CHANGEin the commit footer)What changed
Testing done
Related issues
Closes #
Checklist
closes #702