denshoproject / ddr-cmdln

Command-line tools for automating the Densho Digital Repository's various processes.
Other
0 stars 2 forks source link

ddrimport file segment #121

Closed sarabeckman closed 5 years ago

sarabeckman commented 5 years ago

ddrimport file segment error.txt ddr-csujad-30-master-children-20190220.xlsx

I encountered this bug while importing master files for ddr-csujad-30 on my local VM. The CSV I was using was for three Visual History entities and I was adding the master file to the segment.

ddrimport correctly identified the model as segment but then my CSV failed the validation check and listed I didn't have the correct headers (it lists the headers for an entity csv). As you can see in the text document with the error output I used the comman ddrimport file.

I have been able to successfully ingest the files through the webui.

gjost commented 5 years ago

I'm seeing what you report in my local dev environment.

I have been able to successfully ingest the files through the webui.

Do you mean that you have successfully imported the files individually? Or that you used the CSV import feature in the web UI?

Have you guys imported files from similar spreadsheets in the past? Did it work before but something changed, or did it never work?

gjost commented 5 years ago

FWIW when I've tested file importing in the past I've started by exporting a blank CSV and editing that. ddrexport file --blank /var/www/media/ddr/ddr-csujad-30/ /tmp/ddr-csujad-30-files.csv

Can you give that a try and let me know if it works for you?

gjost commented 5 years ago

Fixed as of commit 1ce0967.