search

sunhater / kcfinder

KCFinder web file manager
http://kcfinder.sunhater.com
402 stars 209 forks source link

Laravel Administrator integration #56

Open frzsombor opened 10 years ago

frzsombor commented 10 years ago

Hi!

First, I would like to thank you this cool project, as I found it really useful in many-many situations.

Currently I'm working on a project with Laravel, and I'm using Laravel Administrator for managing the website. The only thing I really miss from Laravel Administrator is your kcfinder, so I started working on implementing it. I've already worked a couple of days on this, and now I can successfully configure ckfinder from L.A. I'm really close to finishing the integration, so I would like to know if I submit a PR with an integration file for L.A., will you please merge it in the current project?

Thanks in advance!

sunhater commented 10 years ago

Of course...

frzsombor commented 10 years ago

I'm finished with the integration, so you can check the integration file here (not on the commit above as I've already made a commit since then). Currently everything works fine, but I have to talk with the owner of LA before I can create a PR, to minimize the possibility of post-merge modifications. Also please take a look into it and let me know if you want me to modify anything (even in comments).

By the way, I've found something interesting in the integration files, but it's a bit off topic here, so I've opened a new issue (#57) to discuss.

frzsombor commented 10 years ago

@sunhater Ok, it looks like we are good to go! PR submitted, waiting for merge so we can install your package with composer, and don't have to copy all files manually. Thanks in advance!

alsemany commented 9 years ago

hmmm , I really would like to thank you both, for the CKFinder and for the integration Laravel Administration my regards