Skip to content

Fix CSV export/import for Numo-specific fields (SATS, VAT, Inventory)#219

Merged
a1denvalu3 merged 2 commits intomainfrom
fix/import-export-roundtrip
Mar 12, 2026
Merged

Fix CSV export/import for Numo-specific fields (SATS, VAT, Inventory)#219
a1denvalu3 merged 2 commits intomainfrom
fix/import-export-roundtrip

Conversation

@a1denvalu3
Copy link
Copy Markdown
Collaborator

Summary

Merchants reported that importing an exported catalog on a new device lost Numo-specific data, falling back to fiat/disabled settings.
This PR fixes the data loss by expanding the CSV structure while maintaining full retro-compatibility with the existing Square-like template format.

  • Extended CSV Format: Added 4 new columns at the end of the template (Price Type, Price Sats, VAT Enabled, VAT Rate) to preserve Satoshi pricing and tax settings.
  • Inventory Tracking: Correctly maps trackInventory to the Stock Enabled column so it isn't lost.
  • Backwards Compatibility: Old backups can still be imported, correctly defaulting to Fiat/VAT disabled.
  • Testing: Added robust unit tests to ensure ItemManager can successfully roundtrip custom data types through the CSV format.

@a1denvalu3 a1denvalu3 merged commit 9ba4030 into main Mar 12, 2026
3 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in Numo Mar 12, 2026
@a1denvalu3 a1denvalu3 deleted the fix/import-export-roundtrip branch March 12, 2026 19:18
Delgado74 pushed a commit to Delgado74/Numo_internacionalizacion that referenced this pull request Mar 15, 2026
…cashubtc#219)

* fix: CSV export/import roundtrip for Numo fields (SATS, VAT, Inventory)

* fix: allow all mime types for CSV import to prevent grayed out files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant