Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Reporting: Move all CSV exports to endpoint (server) exports #9969

Open
nagpai opened this issue Dec 16, 2024 · 1 comment · May be fixed by #9986
Open

Reporting: Move all CSV exports to endpoint (server) exports #9969

nagpai opened this issue Dec 16, 2024 · 1 comment · May be fixed by #9986
Assignees
@jessy-p
Labels
category: projects For any issues which are part of any project, including bugs, enhancements, etc. focus: reporting status: blocked The issue is blocked from progressing, waiting for another piece of work to be done.

Comments

@nagpai
Copy link
Contributor

nagpai commented Dec 16, 2024 •
edited
Loading

We currently have browser export for list report rows less than 25 ( pagination limit ) and server based export for rows more than that. Changing all exports to server via endpoint will help simplify our CSV exports and manage this functionality at one place.

Acceptance criteria
  • All exports for Transactions, Payouts and Disputes - whether less than 25 rows or more - should be handled via server
  • Tests should be updated to reflect this change
Notes

*paJDYF-g1S-p2
@nagpai nagpai added category: projects For any issues which are part of any project, including bugs, enhancements, etc. focus: reporting labels Dec 16, 2024
@nagpai nagpai mentioned this issue Dec 16, 2024
Closed
This was referenced Dec 18, 2024
Closed
Merged
@nagpai nagpai self-assigned this Dec 18, 2024
@jessy-p jessy-p linked a pull request Dec 18, 2024 that will close this issue
Open
6 tasks
@nagpai nagpai assigned jessy-p and unassigned nagpai Dec 18, 2024
@nagpai nagpai added the status: blocked The issue is blocked from progressing, waiting for another piece of work to be done. label Dec 19, 2024
@nagpai
Copy link
Contributor Author

nagpai commented Dec 19, 2024

Adding blocked status for now since we are not merging changes related to this issue until we have had adequate feedback on the project thread.

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

@jessy-p jessy-p

Labels
category: projects For any issues which are part of any project, including bugs, enhancements, etc. focus: reporting status: blocked The issue is blocked from progressing, waiting for another piece of work to be done.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove browser based CSV export, default to async server-side export Automattic/woocommerce-payments
2 participants
@nagpai @jessy-p