search

OdyseeTeam / odysee-frontend

The code that runs odysee.com
https://odysee.com
MIT License
213 stars 68 forks source link

Transaction Export Broken #2222

Open RauchenwaldC opened 2 years ago

RauchenwaldC commented 2 years ago

Bug The CSV transaction export has invalid columns if any of the videos I uploaded to my channel contains a coma (,) in the title since it is then considered a delimiter.

To Reproduce Steps to reproduce the behavior:

  1. Log in and click on your wallet in the top right
  2. Select the "Transactions" tab
  3. Click "Export" and download the CSV file
  4. Import the CSV file in any program that can read it or at Google Sheets

Expected behavior As a short-term solution, fix the export to encapsulate each value in double-quotes "value 1","value 2",... to prevent this from happening.

Even better would be a proper export that uses the structure in Koinly's Universal import template. See https://docs.google.com/spreadsheets/u/5/d/1dESkilY70aLlo18P3wqXR_PX1svNyAbkYiAk2tBPJng/edit?usp=sharing

Screenshots

Bug

System Configuration IMHO not relevant for this issue.

Additional context Add any other context about the problem here.

tzarebczan commented 2 years ago

Thanks for the report!

I think if we wrapped the names with double quotes, this should fix the issue.