Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bulkrax cannot import a csv file attachment #989

Open
1 task
laritakr opened this issue Dec 5, 2024 · 0 comments
Open
1 task

Bulkrax cannot import a csv file attachment #989

laritakr opened this issue Dec 5, 2024 · 0 comments

Comments

@laritakr
Copy link
Contributor

laritakr commented Dec 5, 2024 •
edited
Loading

Story

When importing a zipped csv with attached files:

  • the csv parser breaks if attached files contain csv files
  • bulkrax ignores zipped csv file attachments, but the work will create and attach any other files

Acceptance Criteria

  • Bulkrax can import an attached csv fileset

Notes
@ShanaLMoore ShanaLMoore mentioned this issue Jan 14, 2025
Open
1 task
@jillpe jillpe mentioned this issue Jan 14, 2025
Closed
1 task
kirkkwang added a commit that referenced this issue Jan 21, 2025
@kirkkwang
🐛 Fix importing CSV as attachment
9bfd8d8 
Prior to this commit, if an importer has a CSV file as an attachment,
the importer would fail because it would try to parse the attached CSV
as the importer CSV.  This commit will add a condition to only look for
a CSV file that is NOT in the `/files` directory.

Ref:
- #989
@kirkkwang kirkkwang mentioned this issue Jan 21, 2025
Merged
kirkkwang added a commit that referenced this issue Jan 21, 2025
@kirkkwang
🐛 Fix importing CSV as attachment
e919617 
Prior to this commit, if an importer has a CSV file as an attachment,
the importer would fail because it would try to parse the attached CSV
as the importer CSV.  This commit will add a condition to only look for
a CSV file that is NOT in the `/files` directory.

Ref:
- #989
kirkkwang added a commit that referenced this issue Jan 21, 2025
@kirkkwang
🐛 Fix importing CSV as attachment
618345e 
Prior to this commit, if an importer has a CSV file as an attachment,
the importer would fail because it would try to parse the attached CSV
as the importer CSV.  This commit will add a condition to only look for
a CSV file that is NOT in the `/files` directory.

Ref:
- #989
kirkkwang added a commit that referenced this issue Jan 21, 2025
@kirkkwang
🐛 Fix importing CSV as attachment
0de575d 
Prior to this commit, if an importer has a CSV file as an attachment,
the importer would fail because it would try to parse the attached CSV
as the importer CSV.  This commit will add a condition to only look for
a CSV file that is NOT in the `/files` directory.

Ref:
- #989
kirkkwang added a commit that referenced this issue Jan 22, 2025
@kirkkwang
🐛 Fix importing CSV as attachment (#1009)
a751f48 
Prior to this commit, if an importer has a CSV file as an attachment,
the importer would fail because it would try to parse the attached CSV
as the importer CSV.  This commit will add a condition to only look for
a CSV file that is NOT in the `/files` directory.

Ref:
- #989
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@laritakr