DataJuggler / BlazorFileUpload

This was originally a wrapper of Steve Sanderson's BlazorFileInput. It has since been upgraded to a wrapper of ASP.NET CORE FileInput
32 stars 11 forks source link

Doesn't support multiple files? #10

Open QadiymStewart opened 4 years ago

QadiymStewart commented 4 years ago

Does this support uploading multiple files at once? "multiple" html input tag like the original from steves files input

DataJuggler commented 4 years ago

No, to answer your question. I guess it is a matter of personal preference, but I like to start as simple as possible at first or the risk of 'what can go wrong' increases as you add more moving parts.

If you are wanting this feature and / or volunteering if you feel you are up for it, I will update the Nuget package. I saw Steve's example with the multiple files once, just never dove in and tried it.

Besides the multiple parameter, do you happen to know if more than one file stream is returned, is that how he does it? I just worked on the code a few days ago and added a SaveToDisk parameter, and if false the MemoryStream is returned in UploadFileInfo, or are more than one UploadFileInfo instances returned?

I tend to ask questions when I could I could just run the program and find out in almost the same time.

I will try and work on this over the weekend.

Thanks, for the feedback.

Corby

araujofrancisco commented 3 years ago

I did an initial implementation for multiple files, so far it seems to be working. You are welcome to test it and report any issues.

DataJuggler commented 3 years ago

I will test it when I get a chance.

I wrote BlazorFileUpload for my site https://pixeldatabase.net , and I only upload 1 file at a time, but I know others will appreciate it.

I have some plans for other sites this would come in handy.

It has been on my to do list to build a progress bar, so I will work on that soon.

Thanks,

Corby

From: Francisco Araujo notifications@github.com Sent: Monday, March 8, 2021 8:33 AM To: DataJuggler/BlazorFileUpload BlazorFileUpload@noreply.github.com Cc: Corby Nichols corby@datajuggler.com; Comment comment@noreply.github.com Subject: Re: [DataJuggler/BlazorFileUpload] Doesn't support multiple files? (#10)

I did an initial implementation for multiple files, so far it seems to be working. You are welcome to test it and report any issues.

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/DataJuggler/BlazorFileUpload/issues/10#issuecomment-792794388, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGI4P7NL2IYROHCFBYGYTI3TCTNZJANCNFSM4K5ZY3IA.

HoneyTauOverTwo commented 3 years ago

I did an initial implementation for multiple files, so far it seems to be working. You are welcome to test it and report any issues.

Thanks for the fork dude! I tried to implement it myself in a similar way to yours but kept getting exceptions for some odd reason. You really saved my skin hahaha

araujofrancisco commented 3 years ago

I did an initial implementation for multiple files, so far it seems to be working. You are welcome to test it and report any issues.

Thanks for the fork dude! I tried to implement it myself in a similar way to yours but kept getting exceptions for some odd reason. You really saved my skin hahaha

You are welcome! Let me know if you find out any issues.