Skip to content

feat(exports): add XLSX exports#1524

Open
anouillz wants to merge 1 commit into
rero:stagingfrom
anouillz:goa-exports-xlsx
Open

feat(exports): add XLSX exports#1524
anouillz wants to merge 1 commit into
rero:stagingfrom
anouillz:goa-exports-xlsx

Conversation

@anouillz

Copy link
Copy Markdown
Contributor

* Adds XLSX files as an export option.
* Relies on: rero/rero-ils#4181
* Closes rero/rero-ils#4168

Co-Authored-by: Ana Gomes <anagoncalves7@icloud.com>
@anouillz
anouillz marked this pull request as ready for review July 22, 2026 18:54
@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2cfbd2bd-0799-4856-af3c-5617b8db3819

📥 Commits

Reviewing files that changed from the base of the PR and between 5abd9d9 and 5ad0627.

📒 Files selected for processing (6)
  • projects/admin/src/app/acquisition/routes/accounts-route.ts
  • projects/admin/src/app/acquisition/routes/orders-route.ts
  • projects/admin/src/app/routes/issues-route.ts
  • projects/admin/src/app/routes/items-route.ts
  • projects/admin/src/app/routes/loans-route.ts
  • projects/admin/src/app/routes/patron-transaction-events-route.ts

Walkthrough

Six administrative routes now include an XLSX export format alongside their existing CSV format: acquisition accounts, acquisition orders, issues, items, loans, and patron transaction events. Each XLSX option uses the corresponding existing export endpoint and sets disableMaxRestResultsSize: true.

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning Most requested export areas were updated, but stats and librarian stats are not covered in the shown changes. Add XLSX export support for the stats and librarian stats CSV export points as required by #4168.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the main change: adding XLSX exports.
Description check ✅ Passed The description is directly related to the XLSX export additions.
Out of Scope Changes check ✅ Passed The changes stay focused on adding XLSX export formats to existing export routes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install timed out. The project may have too many dependencies for the sandbox.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

Exports: add XLSX exports as well as CSV

2 participants