Flagrow image uploader for Flarum forum messages

luceos

In order to conform to best practices, we renamed the package to flagrow/flarum-ext-image-upload please update your composer.json files. The old (non-prefixed) package is marked as abandoned.

Hiob

Super! 🙂

luceos

I just released 0.1.3 which now runs on the new package name (flagrow/flarum-ext-image-upload). This fixes the upload url and some other things.

I'll moderate all issues solved in this thread.

HavocK

luceos You update faster than i install 😃
Great job !

luceos

HavocK that's one benefit of working together in Flagrow with matpompili ; a shared responsibility of maintaining quality packages!

erickoh

Thanks! I've been waiting for an image attach extension since the beginning.
Local storage and resize works.
Imgur gives an error on upload though. I didn't configure an imgur client-id, is that required?

luceos

erickoh yes that's required if choosing imgur.

We're closing in on release 0.2.0 which is a major overhaul but still features the same upload services.

matpompili

0.2.0-beta is out!

New in this version:

permission to allow uploading only to a specific group
requires flarum/composer-installer 0.3.0
moved logic of uploading to php
added/restored imgur and local adapters
now validates image uploads
added cloudinary.com adapter

Please feel free to send PR for translations! Also any feedback is appreciated, as always 😃

luceos

With a big thanks to @Toby and @Franz for providing some great feedback. 0.2.0 reflects a lot of their feedback.

Please also note for the cloudinary upload method to be selectable you need to run the following composer command in the Flarum installation directory:

composer require cloudinary/cloudinary_php

Cheun

Image upload? Looks like normal Markdown to me. Unless you're uploading the image to the local server and it's entering the image files location as the url... .-.

luceos

Cheun related to your feedback, I updated the OP and the readme on github. When you enable the extension a new button is added to the post composer. If you click it and select an image the image will be uploaded to the configured service - by default this is local storage aka your own Flarum installation. Once the upload is done a markdown image is added linking to the newly uploaded image.

luceos

Released 0.2.0-beta.2 fixing a serious issue where creating discussions was impossible due to a js error.

JRay

Great addition!

The normal local upload works great. But I'm having an issue when I enable image resize and ONLY set the max width. It doesn't upload the image and puts a broken reference into the post.

Works fine if BOTH max width and height are set.

matpompili

JRay Yes, you must set both right now.

JRay

matpompili Not a problem. Just letting you guys know. Thanks!

luceos

I was under the impression the dimensions would fallback on 100, I noticed the database stores 0 as default aha.

sogge

Ohai.

Whenever i'm uploading an image using your extension (v0.1.5), it just creates a 0 byte file in the /assets/images folder. I'm using v0.1.0-beta.4 with a little force-https addition in the htaccess:

RewriteEngine On RewriteCond %{HTTPS} !=on RewriteCond %{ENV:HTTPS} !=on RewriteRule .* https://%{SERVER_NAME}%{REQUEST_URI} [R=301,L]

matpompili

sogge Could you please check the apache error log? (or nginx)

luceos

sogge a little force-https addition in the htaccess

Why didn't you change the url in the config.php if I may ask?

I can imagine the POST information won't be send through the 301 redirect.

sogge

matpompili Hm, seems to be a cache problem. Reinstalling the extension helped.

luceos A https-url in the config.php doesn't deny http-access.

« Previous Page Next Page »