s9e / flarum-ext-autoimage

s9e\autoimage is a Flarum extension that automatically turns image links into images.
MIT License
12 stars 2 forks source link

Update for Flarum beta 8 #1

Closed dsevillamartin closed 6 years ago

dsevillamartin commented 6 years ago

Changes made:

Screenshots:

image

JoshyPHP commented 6 years ago

Thanks for the heads-up but I'm going to hold off for the next public release and update all of my extensions at once. It should happen shortly after the next version is officially released.

luceos commented 6 years ago

@JoshyPHP Out of curiosity could you explain the reason for closing this PR? Wouldn't it help you later to merge this and move from there? David spend valuable time preparing this PR, just don't want it to go to waste.. 👍

JoshyPHP commented 6 years ago
  1. I can't test it on my installation.
  2. The whitespace doesn't match the style of the extension so I'd have to edit it.
  3. I have to replicate the changes to the other extensions so I'd rather copy/paste the same code.

Whatever time was spent on this PR isn't necessarily wasted as I'll come back to this PR later to check out which parts of the API change between 0.1.0-beta7.1 and 0.1.0-beta8.

JoshyPHP commented 6 years ago

On that note, I should mention that anybody is welcome to create and maintain replacements for my extensions. Each of them is effectively 1 line of code to enable the relevant plugin (plus some boilerplate code) and I don't have plans on adding more features.

@datitisev If you want to create a replacement under your own namespace (iow not under s9e) please go ahead. If you tag me I'll subscribe to the relevant threads/repositories.

luceos commented 6 years ago

See https://github.com/FriendsOfFlarum/formatting