[UXIT-3811] Add CSV export to tables #212
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
This PR adds CSV export functionality to the service providers table, enabling users to download provider data with a timestamped filename for offline analysis. The implementation introduces a new export button, mapping utilities for data transformation, and refactors shared constants.
Changes:
- Added CSV export button with download functionality using react-csv library
- Created utility function to map provider data to CSV format
- Refactored CURIO_GITHUB_URL constant to shared location and created useExplorerUrl hook
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
| src/components/SoftwareVersion.tsx | Refactored to use centralized CURIO_GITHUB_URL constant |
| src/components/ProviderOverview.tsx | Refactored to use new useExplorerUrl hook for explorer URL retrieval |
| src/app/service-providers/utils/mapProviderToCsvRow.ts | New utility function to transform provider data to CSV format (contains critical bugs) |
| src/app/service-providers/hooks/useExplorerUrl.ts | New custom hook to encapsulate explorer URL logic |
| src/app/service-providers/constants/providers.ts | New constants file with centralized CURIO_GITHUB_URL |
| src/app/service-providers/components/ServiceProvidersTable.tsx | Integrated CSV export functionality into table component |
| src/app/service-providers/components/ExportToCsvLink.tsx | New component rendering CSV download link |
| package.json | Added react-csv, prop-types, and @types/react-csv dependencies |
| package-lock.json | Lock file updates for new dependencies |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@barbaraperic : A couple of things:
|
Thank you! |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 8 out of 10 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…e unused explorer URL dependency
…ity, update structured data utility, and enhance service status capitalization
…V data preparation; remove unused capitalize utility
…eMemo and removing unnecessary state management
…d improve code organization; add new constants file for GitHub URLs
…nce layout responsiveness and simplify CSV export link display
barbaraperic
force-pushed
the
bp/add-csv-export-to-tables
branch
from
992ff4c to
685d4d5
Compare
📝 Description
This PR adds CSV export functionality to the service providers table, allowing users to download provider data for offline analysis.
See Issue #185
🛠️ Key Changes
service-providers-YYYY-MM-DD.csv)Added Dependencies:
react-csvfor CSV generationprop-types(peer dependency for react-csv)@types/react-csvfor TypeScript support📌 To-Do Before Merging
📸 Screenshots
🔖 Resources
react-csv NPM library