Skip to content

Conversation

@pawelwlod
Copy link

Throw an error to the user if a field name heading is blank, alongside the column number in the CSV file being imported.

@pawelwlod pawelwlod changed the title Added appropriate error for blank field name heading in record imports Added appropriate error for blank field name heading in record imports (D1099) Dec 17, 2025
@abeverley
Copy link
Contributor

Thanks Pawel, this is good. I realise the ticket says that this will be a manual test, but it would be relatively easy to add an automated test. Maybe you can take a look at that? A new section in t/014_import.t, creating a new GADS::Import object with appropriate CSV content and wrapping in a try {} block should suffice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants