spree-contrib / spree_editor

Rich text editor for Spree with Image and File uploading in-place.
http://guides.spreecommerce.org
BSD 3-Clause "New" or "Revised" License
112 stars 183 forks source link

WYMEditor doesn't work in production mode #10

Closed romul closed 11 years ago

romul commented 12 years ago
ActionView::Template::Error (admin/wymeditor/jquery.wymeditor.min.js isn't precompiled):
1: <%= javascript_include_tag 'admin/wymeditor/jquery.wymeditor.min.js' %>
2: 
3: <script type="text/javascript">
4:   $(function() {
ehoch commented 12 years ago

I can't get my WYMEditor to even after adding: 'admin/wymeditor/**' to my assets precompile. I get a javascript error of

Uncaught TypeError: Cannot call method 'replace' of undefined jquery.wymeditor.min-f0d3995d6771e2de9df76b1c73b3d173.js:1

It does work for me locally, however... What can I provide to help debug?

lamguy commented 12 years ago

I am getting the same error. In development mode, it doesn't pull out the error but WYMEditor doesn't load.

reelman commented 11 years ago

Same for me.

reelman commented 11 years ago

... but if you use CKEditor https://github.com/secoint/spree_editor it runs just perfect.

JDutil commented 11 years ago

Since this has been outstanding for over a year, and there does not appear to be any interest by people who were using the editor to fix it I'm removing support for WYMEditor, and YUI Editors completely since it's only cruft at this point.

I'm about to push an update that includes #28 to add CKEditor support, which appears to be much better as it allows file uploads etc... There is also still the TinyMCE editor as well if your looking for something a little lighter.

I believe we should cut down on how many editors are actually supported to make the library easier to maintain. If theres a strong will to re-add either of those editors someone who wants to maintain them will need to submit a new tested patch.