Keystone-classic: Unable to upload images, tinymce not rendering

Created on 3 Mar 2015  Â·  9Comments  Â·  Source: keystonejs/keystone-classic

There appears to be a change since 0.3.1 that is stopping tinymce working when images are enabled

tinymce is trying to load /keystone/js/lib/tinymce/plugins/uploadimage/plugin.min.js which doesn't exist in the structure (404), it looks like it can be found in /keystone/js/lib/tinymce/plugins/image/plugin.min.js

Checking through the file the HtmlField type refers to uploadimage but looks like it should be "image"

I manually changed it locally to test if it would fix but even after changing the upload dialog only shows a source field without any upload functionality

Most helpful comment

@JedWatson Looks like this issue is back with 4.0.0. Can we add this TinyMCE plugin back to master or is there another preferred way to do image uploading in the new version?

All 9 comments

+1 confirmed here.

It seems the whole uploadimage/ plugin dir has been removed. https://github.com/keystonejs/keystone/commit/dbb1594eb2f63709f9f582b74fc3a69dbd304ef5

+1

having the same experience trying to use wysiwyg cloudinary images for tinyMCE on 0.3.2.

a temporary hack would be to host the images elsewhere and add that manually into tinymce via the code button. Like <img src="where/ever/you/saved/it.jpg" />.

I've submitted a pull request that adds the plugin back in. This looks like it resolves the issue!

Cheers @pswoodworth, appreciate the PR! merged now, will be released in the next version (soon!)

@JedWatson you got it – Keystone has really been a fantastic platform for us. Thanks for all your work – I'd write the heck out of a testimonial if you ever want one!

:D

@JedWatson Looks like this issue is back with 4.0.0. Can we add this TinyMCE plugin back to master or is there another preferred way to do image uploading in the new version?

@ehoch

I added the uploadimage plugin to my fork of keystone. https://github.com/svpe/keystone

I've sent a pull request. Depending on if the plugin was deliberately left out in this version or not @JedWatson will pull it. However in my use case everything seems to work just fine now.

https://github.com/keystonejs/keystone/pull/4232

Was this page helpful?
0 / 5 - 0 ratings

Related issues

josephg picture josephg  Â·  4Comments

christroutner picture christroutner  Â·  4Comments

koenoe picture koenoe  Â·  4Comments

webteckie picture webteckie  Â·  5Comments

sorryididntmeantto picture sorryididntmeantto  Â·  3Comments