search

ESI-FAR / INA-tool

A tool for visualising and analysing Institutional Statements
MIT License
2 stars 0 forks source link

Improve CSV parsing #64

Open sjvrijn opened 2 months ago

sjvrijn commented 2 months ago

We currently parse the CSV files manually, by splitting on newlines and comma's. This means we don't detect or support alternative separators, and might run into trouble if users include newlines in their cell contents.

We should look into using a standard library for our CSV parsing instead to save us the maintenance and any other (unforseen) issues

suvayu commented 2 months ago

Since we provide an Excel template, we should probably also accept Excel files.

Since the expectation is that people use the template, I think it should be fine to reject files that don't conform to the template.