Skip to content

fix: rename misleading USD Equivalent column in CSV exports - #21

Merged
ibrahimmosouf-png merged 1 commit into
OrbitChainLabs:mainfrom
postqcoin:fix/csv-usd-equivalent-placeholder
Jun 18, 2026
Merged

fix: rename misleading USD Equivalent column in CSV exports#21
ibrahimmosouf-png merged 1 commit into
OrbitChainLabs:mainfrom
postqcoin:fix/csv-usd-equivalent-placeholder

Conversation

@postqcoin

Copy link
Copy Markdown
Contributor

Problem

The 'USD Equivalent' column in donation CSV exports is hardcoded to '0.00' with a comment indicating it would be fetched from a price cache in production. This is misleading for users who may trust the value for tax/accounting purposes.

Fix

  • Renamed column to 'USD Equivalent (pending)' to clearly indicate the value is not yet available
  • Changed hardcoded '0.00' to 'N/A' with explanatory comment
  • Applied to all three export locations:
    • src/users/users.service.ts
    • src/users/export.processor.ts
    • src/donations/donations.service.ts

Closes #15

The 'USD Equivalent' column in donation CSV exports was hardcoded to
'0.00' with a comment indicating it would be fetched from a price cache
in production. This is misleading for users who may trust the value
for tax/accounting purposes.

Changes:
- Renamed column to 'USD Equivalent (pending)' to clearly indicate
  the value is not yet available
- Changed hardcoded '0.00' to 'N/A' with explanatory comment
- Applied to all three export locations:
  - src/users/users.service.ts
  - src/users/export.processor.ts
  - src/donations/donations.service.ts

Closes OrbitChainLabs#15

@ibrahimmosouf-png ibrahimmosouf-png left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice honest fix, @postqcoin 👏

Renaming the column to "USD Equivalent (pending)" and replacing the placeholder 0.00 with N/A plus a short comment is exactly the right call until the price oracle lands — much better than shipping misleading zeros. Small, scoped, and closes #15 cleanly.

Merging now. Thanks for taking this on!

@ibrahimmosouf-png
ibrahimmosouf-png merged commit 7eee2cb into OrbitChainLabs:main Jun 18, 2026
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.

[MEDIUM] — CSV donation export hardcodes "USD Equivalent" = 0.00; downstream consumers may trust the column for tax / accounting workflows

3 participants