safe-airdrop icon indicating copy to clipboard operation
safe-airdrop copied to clipboard

Unresponsive File Upload & Drag+Drop

Open bh2smith opened this issue 2 years ago • 12 comments

Recently we have had a user (Chrome on Windows) who cannot upload the sample file:

When dragging the file in the drop box is highlighted in red indicating that the app already knows the file will not be supported.

Screenshot 2021-11-17 at 10 33 07 AM

I can only assume that the file was somehow changed by the OS after download. We have tried manually altering the file extension to be explicitly csv. but this may have resulted in bein renamed to sample.csv.whatever_ms_added_cuz_it_sucks. It is unclear if the actual file extension was modified by the operating system (since it doesn't display file extensions in the file navigator).

bh2smith avatar Nov 17 '21 10:11 bh2smith

I'm also having this issue. I thought it was a chrome/chromium issue too but also tried firefox with the same result. I was using Brave - put the shields down. Same result. I made sure the extension of the file is .csv with no extra file extension.

billyjitsu avatar Dec 10 '21 01:12 billyjitsu

@billyjitsu could you upload the file which is not working here so we can try to reproduce the issue?

schmanu avatar Dec 10 '21 04:12 schmanu

wXdai test.csv

Sure thing

billyjitsu avatar Dec 10 '21 04:12 billyjitsu

One thing to note, I'm on windows 10... tried using brave, chrome and firefox .. if that helps

billyjitsu avatar Dec 10 '21 04:12 billyjitsu

I am beginning to think this might be a windows thing (both cases so far have been there) and I none of us have ever tried this there.

@billyjitsu could you also confirm that the same thing happens to you with the sample transfer file?

bh2smith avatar Dec 10 '21 08:12 bh2smith

I forgot to write it here. It was related to the .CSV file being created with excel. It was not comma separated but tab separated and was missing a column. I suggested to copy paste the file content as a workaround which helped. After my vacation I can try to reproduce this with a windows machine running excel and add a section into the FAQ if needed.

schmanu avatar Dec 10 '21 13:12 schmanu

I wonder if we should try to support multiple "spreadsheet" formats.

bh2smith avatar Dec 10 '21 13:12 bh2smith

This definitely appears to be a windows issue. I have another user reporting the same problem.

bh2smith avatar Dec 14 '21 10:12 bh2smith

This definitely appears to be a windows issue. I have another user reporting the same problem.

Do you happen to have the file the user used? Or the workflow step by step? Was the file created in excel?

schmanu avatar Dec 14 '21 15:12 schmanu

So to follow up on my issue. I believe it to be a pure being associated with MS excel issue. I'll create a .csv file in "notepad" and associate the files with notepad is no issue. I associate the .csv file with excel (opens with excel by default) and then I have issues. I have to open with notepad, copy and paste for it to work.

billyjitsu avatar Dec 14 '21 16:12 billyjitsu

This definitely appears to be a windows issue. I have another user reporting the same problem.

Do you happen to have the file the user used? Or the workflow step by step? Was the file created in excel?

I think, even if you save the sample file to a windows machine it is being converted in some way to an MS format of some sort. I have dealt with issues of this sort in the past back in my python days and remember a difference between .xls and .xlsx file formatting. One of them is a proprietary format that can't be opened and parsed with any standard libraries while the other is just a zip file in disguise that unpacks to an xml format. The same might be happening here.

So I would guess that if you save the sample transfer file on a windows machine, even that can not be uploaded to the app.

bh2smith avatar Dec 14 '21 20:12 bh2smith

I think this might have something to do with microsoft CSVs having \r\n for line breaks (i.e. with carriage return) while unix CSVs have \n. Just a thought.

bh2smith avatar Feb 11 '22 13:02 bh2smith