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

[Data Import] Misleading Error Message #11474

Closed
ccrvlh opened this issue Nov 6, 2017 · 3 comments · Fixed by frappe/frappe#4601
Closed

[Data Import] Misleading Error Message #11474

ccrvlh opened this issue Nov 6, 2017 · 3 comments · Fixed by frappe/frappe#4601
Assignees
Labels
Milestone

Comments

@ccrvlh
Copy link

ccrvlh commented Nov 6, 2017

Hi,

I just wanted to suggest to be able to see a more assertive error on importing files.
After a few hours getting the error related to Import Format, I discovered that the error was in the Import Content.

Even though it was my fault, the system made me look for the error in the format of the file (I ended up exporting thousands of .csv encryptions), when in reality it was just a matter of linking fields correctly (content).

Thanks a lot.

@ccrvlh ccrvlh changed the title [Import] Misleading Error Message [DATA IMPORT] Misleading Error Message Nov 7, 2017
@ccrvlh ccrvlh changed the title [DATA IMPORT] Misleading Error Message [Data Import] Misleading Error Message Nov 7, 2017
@umairsy
Copy link
Member

umairsy commented Nov 9, 2017

Hello,

Can you please be more specific about the change required in the format to make it work? It will help us resolve this issue faster.

@umairsy umairsy added the bug label Nov 9, 2017
@ccrvlh
Copy link
Author

ccrvlh commented Nov 9, 2017

I was uploading the file in the correct format CSV, but I was missing the "Parent Group" info for some lines (as they were already parents) - which is a content issue.

The thing is, the template file also has blank Parent Group information for the lines which are also parents, so the process ends up a bit misleading.

It would be great to have more specific warnings:
Eg:
"Missing 'Parent Group' information for line XX"
"Link not found for 'Company' for line XX"
"You have uploaded a XLSX file, please upload a CSV file"
"CSV file not recognized - make sure it is sepparated by comma ','"

(The last one might sound absurd, but in my country, the number format is "." for thousands and "," for decimals and when you export CSV the columns are separated by ";", so you have to manually replace the items - not very easy to find out as is not that obvious that CSV files depend on number format configuration of the system)

@rmehta rmehta added this to the November 2017 milestone Nov 14, 2017
@Zlash65
Copy link
Contributor

Zlash65 commented Nov 22, 2017

"Link not found for 'Company' for line XX" - This type of error for link field is already handled.
"You have uploaded a XLSX file, please upload a CSV file" - Uploading through XSLX is valid.
"Missing 'Parent Group' information for line XX" - Can you provide steps to reproduce this..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants