Skip to content
This repository has been archived by the owner on Jul 10, 2022. It is now read-only.

Updated CKEditor to 4.6.1 #737

Merged
merged 5 commits into from Feb 15, 2017

Conversation

Liongold
Copy link
Contributor

@Liongold Liongold commented Dec 29, 2016

Fixes #723. May also improve the auto-save situation, as described in #725.

Fontawesome support is missing.

@mrusme Does this make the pull request unmergeable?

@Liongold
Copy link
Contributor Author

Liongold commented Jan 2, 2017

Thanks to the new commit added, this also replaces #730.

@Liongold
Copy link
Contributor Author

Liongold commented Jan 3, 2017

Closing and reopening request to generate builds

@kevthehermit
Copy link

Do you know when this will get merged, wanting to update my production instance, but its docker so need to wait for this

@Liongold
Copy link
Contributor Author

@mrusme Do you have any time to check this out please?

@mrusme
Copy link
Member

mrusme commented Jan 20, 2017

@Liongold I'm not 100% sure about this. Have you tested it locally?

@Liongold
Copy link
Contributor Author

@mrusme Yes, I've tested it. If you are uncertain about anything, no problem, I can test it in a better way. However, if it isn't a problem for you, could you take care of the other requests? They should be small updates but are pending for quite some time.

@Liongold
Copy link
Contributor Author

As I've said, the Font Awesome plugin is missing because there does not seem to be a plugin for this version.

@kevthehermit
Copy link

@Liongold @mrusme any movement on this, Or shall i just create my own fork for now.

@Liongold
Copy link
Contributor Author

Liongold commented Feb 7, 2017

@kevthehermit I don't have any more information to share on this.

@mrusme mrusme merged commit 6d5729d into paperwork:develop Feb 15, 2017
@mrusme
Copy link
Member

mrusme commented Feb 15, 2017

@kevthehermit feel free to fork -> implement -> PR. :-) 👍

@Liongold
Copy link
Contributor Author

Thanks @mrusme for taking a look at all the pending requests.

@mrusme
Copy link
Member

mrusme commented Feb 15, 2017

@Liongold sure thing, sorry for the delay

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

Successfully merging this pull request may close these issues.

None yet

3 participants