stephskardal / rails_admin_import

Rails Admin Import functionality
http://www.endpoint.com/
MIT License
160 stars 122 forks source link

question/discussion: option for blank rows in csv #86

Closed hstoebel closed 6 years ago

hstoebel commented 7 years ago

Not sure if this is already possible and if not if it makes sense to implement. What if users could optionally check a box to indicate that their .csv file will contain blank rows. If that box is checked, blank rows are skipped. Maybe the first row would always need to have headers.

In a demo today, one of my users suggested that it would be useful for her workflow if she could visually separate groups of data using blank lines.

Apologies if this is already possible.

monkbroc commented 7 years ago

You could do this by modifying the CSV parser. Just don't yield the line when it is blank.

On Feb 23, 2017 4:19 PM, "Jacob Stoebel" notifications@github.com wrote:

Not sure if this is already possible and if not if it makes sense to implement. What if users could optionally check a box to indicate that their .csv file will contain blank rows. If that box is checked, blank rows are skipped. Maybe the first row would always need to have headers.

In a demo today, one of my users suggested that it would be useful for her workflow if she could visually separate groups of data using blank lines.

Apologies if this is already possible.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/stephskardal/rails_admin_import/issues/86, or mute the thread https://github.com/notifications/unsubscribe-auth/AB1KwadfZzgwvqWjRsfCcF2LKSds13Q5ks5rfff9gaJpZM4MKg0- .

monkbroc commented 6 years ago

Blank lines will now be skipped