CSVs with BOM might not be importable

NBS-LLC / ng-spotify-importer

Import playlists into Spotify.

https://nickwanders.com/projects/ng-spotify-importer/

GNU General Public License v3.0

100 stars 17 forks source link

CSVs with BOM might not be importable #216

Closed NBS-LLC closed 1 year ago

NBS-LLC commented 1 year ago

The community reported that CSV files with a BOM header don't import properly. This might have been fixed in 1.4.0 when the parsing code was improved. This needs to be confirmed.

More details can be found in the original post.

NBS-LLC commented 1 year ago

Created the following sample playlist with BOM: Screenshot 2023-05-08 9 52 01 PM

Which imported properly in v1.4.1:

Was not able to reproduce the bug.

NBS-LLC commented 1 year ago

The CSV parsing library being used handles CSVs with BOMs: https://github.com/mholt/PapaParse/pull/961

It's possible that the original issue reported by the community was the symptom of another issue. More details are needed. Closing this bug as invalid.