@clarkwinkelmann
New version is working well so far. The uploading indicactor is definitely a welcomed feature 🙂 Thanks!
I'm posting here as a heads-up: There's an issue with the CDN prefixing. I commented on this PR here:
FriendsOfFlarum/upload184
I'm pretty sure that the PR will fix the CDN issue but it's being held up by a code-style issue. (One that doesn't seem to make sense considering the way the array helper (Arr::get) works. More info on the PR comments.)
An extra reason why this is important is because Amazon's SSL certificate for *.s3.amazonaws.com is not working. Which means that the way that the Upload extension constructs the URL, results in images not loading. (Browser blocks them because they're not secure. This is if you're hosting your forum on HTTPS)
One workaround to this issue is to use the CDN prefix setting, because then the URLs can be written to use this syntax:
https://s3.amazonaws.com/bucket-name/path-to-file
Instead of:
https://bucket-name.s3.amazonaws.com/path-to-file
I'm hoping that PR can be merged because it fixes both issues at once. Thanks for reading!