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

File upload for importing partners is broken #532

Open
benoitdevrieze opened this issue Mar 14, 2016 · 4 comments
Open

File upload for importing partners is broken #532

benoitdevrieze opened this issue Mar 14, 2016 · 4 comments
Labels
bug Something isn't working level: medium

Comments

@benoitdevrieze
Copy link

This feature:

screenshot 2016-03-14 at 09 52 24

Gives this unexpected output:

screenshot 2016-03-14 at 09 48 51

@benoitdevrieze
Copy link
Author

Update: The first of the data was processed correctly up until a record were not all field contained a value. The software probably throws on an empty value.

Graceful failure needed!

@TomNaessens
Copy link
Member

Can you show an excerpt of the CSV line? (Made anonymously of course)
On 14 Mar 2016 10:01, "Benoit De Vrieze" [email protected] wrote:

Update: The first of the data was processed correctly up until a record
were not all field contained a value. The software probably throws on an
empty value.

Graceful failure needed!


Reply to this email directly or view it on GitHub
#532 (comment).

@benoitdevrieze
Copy link
Author

Will do.

Additional errors:

  • Names with accents on the letters crash the processing (just as in the screenshots)
  • Email adress with uppercase characters is rejected (example: [email protected]) (will move this to a new issue)

@benoitdevrieze
Copy link
Author

CSV example

Hugh Develop,[email protected]
Jane Dhoe,
Walter White,[email protected]

Expectation: Correctly process line 1 and 3 and report line 2 as an error for manual fixing
Reality: Correctly processes line 1, throws an error on line 2, never processes line 3

@NuttyShrimp NuttyShrimp added bug Something isn't working level: medium and removed bug labels Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working level: medium
Projects
Status: No status
Development

No branches or pull requests

3 participants