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.

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

Recipient Importer 'Skip first row' bug #101

Open
cadie opened this issue Jan 15, 2021 · 0 comments
Open

Recipient Importer 'Skip first row' bug #101

cadie opened this issue Jan 15, 2021 · 0 comments
Assignees
Labels
bug A problem with existing functionality

Comments

@cadie
Copy link
Contributor

cadie commented Jan 15, 2021

Describe the bug
When importing a csv of recipients, the first row in the csv is skipped even if ‘skip first row’ is left unchecked. The subprocess status count seems to be off as well, upon completion of testing a 3 email csv import it came back as ‘4 of the 3 recipients processed’, even though it only successfully imported two emails.

Steps to Reproduce

  1. Create a CSV where the first row data is an email (no "email" header row)
  2. Import the recipients into a new recipient group, leaving "Skip first row" unchecked
  3. When finished the new group should be missing the first email listed in the csv.

Expected Behavior
If the 'Skip first row' is unchecked, the first row of the CSV should be imported if it's a valid email.

Possible Fix
Ensure that the skip first row flag is respected.

@cadie cadie added the bug A problem with existing functionality label Jan 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A problem with existing functionality
Projects
None yet
Development

No branches or pull requests

4 participants