[Bug] Transactions tab CSV export not migrated to shared csv-export utility — still has token symbol and date format issues

[grantfox-oss]

Getting Started

1. Fork the repository: https://github.com/JointSave-org/Joint_Save
2. Clone your fork:

   git clone https://github.com/<your-username>/Joint_Save.git
   cd Joint_Save

3. Create a new branch:

   git checkout -b fix/transactions-csv-export-consistency

---

Overview

PR #154 added a well-built, well-tested shared CSV export utility (frontend/lib/csv-export.ts) with proper RFC 4180 cell escaping, and used it for the new admin audit log feature. However, the existing Transactions tab export (frontend/components/dashboard/transactions.tsx) still has its own separate, hand-rolled CSV logic that was never migrated to use this new utility — meaning the original issue this work was meant to address (#149) is only partially resolved.

Current Behavior

In transactions.tsx's exportCSV function:

const header = ["Date", "Pool Name", "Pool Type", "Activity Type", "Amount", "Transaction Hash"]
const rows = filtered.map((a) => [
  new Date(a.created_at).toLocaleDateString(),
  a.pool_name ?? "",
  a.pool_type ?? "",
  a.activity_type,
  a.amount != null ? a.amount.toFixed(2) : "",
  a.tx_hash ?? "",
])

Two real problems:

1. No token symbol on amounts. The "Amount" column exports a bare number (e.g. 50.00) with no indication of which token it refers to. Since multi-token support has shipped, a pool using USDC and a pool using XLM would both export indistinguishable numbers, which is genuinely ambiguous when reading the file later, especially across multiple pools with different tokens.
2. Locale-dependent date format. new Date(a.created_at).toLocaleDateString() produces output that varies based on the viewer's browser locale (e.g. 6/27/2026 vs 27/6/2026), which is ambiguous in an exported file that might be opened on a different machine or shared with someone else.

Required Fix

• Migrate transactions.tsx's exportCSV to use buildCsv/downloadCsv from frontend/lib/csv-export.ts, consistent with the admin audit log's usage
• Add a token symbol column (or append it to the Amount column, e.g. "50.00 USDC") so each row's amount is unambiguous
• Replace toLocaleDateString() with an unambiguous, fixed format (e.g. ISO 8601: 2026-06-27), consistent with how dates are already formatted elsewhere in exports

Acceptance Criteria

• transactions.tsx's export uses the shared csv-export.ts utility, not its own separate implementation
• Exported amounts clearly indicate the token symbol per row
• Exported dates use a fixed, locale-independent format
• Existing export functionality (filtering, file download) continues to work correctly
• Manually verify the exported CSV opens correctly in both Excel and Google Sheets with no encoding/escaping issues

[CHKM001]

Would love to work on this and hash out the bug for you
Thanks

[WorldsBestSoftwareDeveloper]

I'd like to work on this issue.

I've recently contributed multiple frontend improvements to this repository, including accessibility enhancements and UI refactors. This issue is a good fit because it focuses on consolidating duplicated logic into an existing shared utility while preserving current functionality.

My approach will be to replace the hand-rolled CSV generation in the Transactions tab with the shared "csv-export" utility, ensure exported dates use a consistent locale-independent format, include the token symbol for each exported amount, and verify that filtering and download behavior remain unchanged. I'll also test the generated CSV to ensure it opens correctly in both Excel and Google Sheets and that the shared export utility continues to handle escaping correctly.

I'll submit a clean, focused PR that satisfies all acceptance criteria and keeps the implementation consistent with the existing shared export infrastructure.

[Yerimahjr]

I recently completed PR #25 on this repo (RFC 4180 CSV export — merged) and PR #104 (API rate limiting). Familiar with the codebase including csv-export.ts and the transactions component.
For this issue I'll migrate transactions.tsx's exportCSV to use buildCsv/downloadCsv from frontend/lib/csv-export.ts, add a token symbol column (or append to Amount as "50.00 USDC"), and replace toLocaleDateString() with ISO 8601 format (2026-06-27), consistent with the audit log export. All existing filtering and download behavior will be preserved.

[grantfox-oss]

🦊 GrantFox — @WorldsBestSoftwareDeveloper has been assigned to this issue as part of the Official Campaign campaign!

Next steps:

1. Open a Pull Request referencing this issue (e.g., Closes #159)
2. Your PR will be reviewed by the JointSave-org maintainers

Good luck! Track your progress on GrantFox.

[grantfox-oss]

🎉 This issue has been marked as completed on GrantFox as part of the Official Campaign campaign!

@WorldsBestSoftwareDeveloper's PR #161 was approved and merged by @Sendi0011.

🏆 @WorldsBestSoftwareDeveloper: You earned 35 FoxPoints for this contribution! Your current tier: Explorer (153 total points). Track your full progress on GrantFox.

👏 Great work, @WorldsBestSoftwareDeveloper! Keep contributing to JointSave-org.