FriendsOfFlarum / upload

The file upload extension with insane intelligence for your Flarum forum.
https://discuss.flarum.org/d/4154
MIT License
177 stars 96 forks source link

Multiple uploads per post #15

Closed alicolville closed 7 years ago

alicolville commented 7 years ago

Would it be possible to allow multiple uploads per post? Would be great if a member could select 10 images at once :)

Cheers! Keep up the great work.

luceos commented 7 years ago

What would be the benefit versus clicking the upload button multiple times, just out of curiosity?

alicolville commented 7 years ago

Just to save them clicking the upload icon 10 times... they could instead click it once... select multiple images in the dialog... or select all... hit upload... All done...

There may not be a massive benefit, but I've already had two people asking for it... I know myself I like to open the dialog... sometimes select all and done...

Thanks again :)

luceos commented 7 years ago

I think this requires a slight modification in the payload send and received and a slight edit to the upload input field.

alicolville commented 7 years ago

Sounds good! :+1:

luceos commented 7 years ago

@yekenuk this feature is available in dev-multi feel free to update your dependency and give it a test drive. I'll ship the feature with the next feature.

alicolville commented 7 years ago

Great! Not being an expert on composer, do I just run?

composer update flagrow/upload

luceos commented 7 years ago

edit your composer.json where it says flagrow/upload change the value after to dev-multi and then run composer update; please note that composer update is not something you want to do in a production/live environment :smile:

alicolville commented 7 years ago

Ah cheers mate...

I'm running flarum in a small production site ha :)

Thanks again! Keep up the great work

luceos commented 7 years ago

Well - between us - you can change the dependency as proposed above, if you encounter issues, change it back and run the command again. Delete any assets/*.js to drop the cache.

alicolville commented 7 years ago

Brilliant thanks.