search

Laravel-Backpack / community-forum

A workspace to discuss improvement and feature ideas, before they're actually implemented.
28 stars 0 forks source link

CkEditor(Wysiwyg) file upload #641

Open izica opened 10 months ago

izica commented 10 months ago

Hello, in backpack/pro, the CKEditor file upload doesn't function as intended. When attempting to upload a file, nothing happens. The console displays the error message "filerepository-no-upload-adapter." I resolved this issue by modifying the "ckeditor.blade.php" file and developing a customized upload adapter. With fix when uploading a file, the file is moved to the "/uploads/ckeditor/{modelName}" directory.

How can I pr this fix?

tabacitu commented 10 months ago

Hi @izica - ouch, I wasn't aware. Thanks for letting us know, and for providing the fix.

It's tricky to create a PR to PRO, since it's closed-source. Can you please send the ckeditor.blade.php blade file to hello@backpackforlaravel.com ? I'll pick it up and create the PR for you.

Thanks a lot!

izica commented 10 months ago

@tabacitu I've sent the message with fix to email.

This issue emerged because CKEditor was upgraded from version 4 to 5, but CKEditor is not fully backwards compatible.

haniramadhan-kkp commented 9 months ago

I amn still waiting for the bug fix? Can I use other template or backpack version to use WYSIWYG editor?

pxpm commented 9 months ago

@haniramadhan-kkp sorry for the time it took me to get back here.

I've got the info needed, and will work on this during this week.

Thanks for your patience.

Cheers

adriallongarriu commented 8 months ago

@pxpm any news on this? my issue(#5323) was closed 3 weeks ago only to move the conversation here...

pxpm commented 8 months ago

Sorry @adriallongarriu since this is in the private repo it seems that we are not moving foward, but we are currently making progress on it, and already have a partial integration working. I think by the end of this week at most we should be able to ship it.

here is the status of it: image

kevinartimedia commented 6 months ago

image image

Hi! How is this topic? It doesn't work for us.

pxpm commented 6 months ago

Hey @kevinartimedia we have this "pretty much ready" in our repo, but we've decided to push the fix for this upstream in the Laravel community package. The reason is that we would need a lot of hackywaky stuff and make people lifes very difficult to upgrade if we push this change in our repo instead of the upstream.

Unfortunately we didn't had the time to finish this, the PR's need need to be refactored so that they can be submitted to the upstream package. This is in my backlog, I just have other priorities come up in the meanwhile. I hope I can give everyone a happy new year gift on this one 🙏

Cheers, and thanks for your patience.

FilippoCalabrese commented 5 months ago

Any update?

zisakmc commented 3 months ago

The error lead me here, hope update available soon

FilippoCalabrese commented 3 months ago

The error lead me here, hope update available soon

Using TinyMCE field type seems to fix the issue. It is not a solution but is a workaround to me 😄

wfgm5k2d commented 1 month ago

This problem still exists in my backpack 6 Any solutions?

pxpm commented 1 month ago

Elfinder was added back to CKEditor on backpack/pro 2.2.1 released today.

The custom uploader (for drag 'n drop and image upload) will follow after we merge https://github.com/Laravel-Backpack/CRUD/pull/5478 that would allow us to create an uploader for ckeditor too.

Please see the updated docs here: https://backpackforlaravel.com/docs/6.x/crud-fields#ckeditor-pro

pxpm commented 1 month ago

I will leave this issue open until we add the Custom Uploader (like we are adding for easymde).