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

Add ckeditor in repository #43

Closed
Wtower opened this issue Mar 18, 2016 · 4 comments
Closed

Add ckeditor in repository #43

Wtower opened this issue Mar 18, 2016 · 4 comments
Milestone

Comments

@Wtower
Copy link
Owner

Wtower commented Mar 18, 2016

CKEditor is used in by 9cms in admin for node editing. CKEditor recommends that the script is pre-configured to use only the necessary script plugins for maximum security.

Add the stripped-down CKEditor version in the repository.

@Wtower
Copy link
Owner Author

Wtower commented Mar 18, 2016

Fixed with b6df0a9

@Wtower Wtower closed this as completed Mar 18, 2016
@Wtower
Copy link
Owner Author

Wtower commented Mar 18, 2016

CKEditor takes up a large part of the repository on the other hand, and makes it too opinionated. Remove from repo and consider different options.

@Wtower Wtower reopened this Mar 18, 2016
@Wtower Wtower added this to the v0.6.0 milestone Mar 19, 2016
@Wtower
Copy link
Owner Author

Wtower commented Mar 30, 2016

Use django-ckeditor.

@Wtower
Copy link
Owner Author

Wtower commented Apr 6, 2016

Package django-ckeditor requires to download ckeditor separately too, so it will not be used after all.

The ckeditor files will be removed from repo and an additional download instruction will be added in readme, along with a sample build-config file.

Wtower added a commit that referenced this issue Apr 6, 2016
* Remove ckeditor from repo except from build-config and update readme #43

* Remove ckeditor files
@Wtower Wtower mentioned this issue Apr 6, 2016
@Wtower Wtower closed this as completed Apr 6, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant