Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redactor 3 #886

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Redactor 3 #886

wants to merge 18 commits into from

Conversation

gr8bit
Copy link
Contributor

@gr8bit gr8bit commented Jul 4, 2019

All tests succeed, even the system/capybara tests (well, most of the time. Racing conditions seem to be racing).

Biggest changes for version 3 of Redactor (please review @GBH, I don't know if this breaks something):

  • lots of additional plugins (5 of them bring a little CSS which I always included because it's so little)
  • language/i18n files have been massively reduced (were the extra ones, e.g. "az", "ba", "bg", user contributed?) and have been partly renamed (e.g. "zh-CN" -> "zh_cn"), bringing them out of sync with the rails locales -> needs work

Let's finalize this together @GBH :) Do you have a real world project we can throw this into?

…a into redactor-3

* 'master' of github.com:bichinger/comfortable-mexican-sofa: (24 commits)
  should not need to manually start service
  updating readme. trying to get travis working againg
  gotta make it available in the list
  fixing problem with seed rake task
  added support for hr locale
  adding readme about version 1 working on rails 5.2
  coppers
  reworked options_for_select a bit
  Fix default classes for comfy cms seeds import and export task (comfy#878)
  Bump comfortable_mexican_sofa to 2.0.18
  fix so ci runs
  files require labels
  cleaning up file label setter a bit
  fixing link to the variant
  bumping travis rubys
  just deleted all lines previously commented out
  fix for variant url
  remove the pry
  remove trailing spaces
  correct typo on function name
  ...
@gr8bit
Copy link
Contributor Author

gr8bit commented Jul 10, 2019

I just noticed this HowTo for a Reactor upgrade:
https://github.com/comfy/comfortable-mexican-sofa/wiki/HowTo:-Upgrading-Redactor-Wysiwyg

I did not see this before, sorry. We'll have to go through that as well I guess.

@gr8bit
Copy link
Contributor Author

gr8bit commented Aug 6, 2019

@GBH fyi: we'll be using it in a project, so we're currently fixing everything that the update broke. :)

@GBH
Copy link
Member

GBH commented Aug 6, 2019

@gr8bit 👍 Rails 6 final release probably will happen this fall. I'll jump in sometimes then to merge everything in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants