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

Ckeditor 4.4.8 and a builder for next releases #44

Closed
wants to merge 11 commits into from
Closed

Conversation

gotcha
Copy link
Member

@gotcha gotcha commented Sep 18, 2015

Upgrade Ckeditor to 4.4.8
Add scripts to automate update of ckeditor

@gotcha gotcha changed the title Builder Ckeditor 4.4.8 and a builder for next releases Sep 18, 2015
@gotcha gotcha mentioned this pull request Sep 18, 2015
@gbastien
Copy link
Member

Hi @gotcha

I tested a little and it seems that some plugins are missing, if I activate every buttons, some do not show like :

  • spellchecker
  • Scayt (scayt is missing in directory "plugins")

Maybe is it no more builtin now?

+++

Gauthier

@gotcha
Copy link
Member Author

gotcha commented Sep 23, 2015

@gbastien
Copy link
Member

Hi @gotcha

actually I prefer you adapt the script ;-), I do not want to spend to much time discovering what it does.

What I just wanted to underline is that full presets release are left available, for example : http://download.cksource.com/CKEditor/CKEditor/CKEditor%204.4.8/ckeditor_4.4.8_full.zip

This way you are sure to get the right full preset, it can evolve (http://ckeditor.com/presets).

Then the builder script could "just" take care to grab extra plugins (or we can do it manually double-checking version compatibility). We need to check the _full preset, especially regarding the default selected skin that seems to be "Moono" instead of "Moono color"...

So :

  • download full preset release;
  • change (download it and patch the build-config.js);
  • download extra plugins manuall...

?
Gauthier

@gotcha
Copy link
Member Author

gotcha commented Sep 23, 2015

I'll look at this when I'll have time.

@gbastien
Copy link
Member

Hi @gotcha

to me a "manual recipe" is not a problem, PR review will cover this.

Downloading the full release will make sure we have relevant things and not create our builder script (if I look here, I do not want : https://github.com/ckeditor/ckeditor-dev/blob/master/dev/builder/build.sh).

We may write a script just receiving the CKEditor version we want and :

We may work on this too if you are ok with this approach.

Gauthier

@gotcha
Copy link
Member Author

gotcha commented Sep 24, 2015

The approach I took is to use ckbuilder from the ckeditor team:

https://github.com/collective/collective.ckeditor/blob/74b979030e706cf942d9304bae1db0df4c9e00da/dev/build-ckeditor/build.sh

The problem I have with doing clicks is that it is too easy to forget steps.

For instance, I had forgotten scayt was important for you and did not check it.

By scripting the release, we will be able to get the fixes from ckeditor quicker as well.

@gbastien
Copy link
Member

@gotcha

I am ok to script, but what I propose is to use the release the CKEditor team builds because we do not know what could change in the future...

Script the fact to get the release, "sed" it to "Moono color", script the the fact to get plugins and to merge it in the release, ...

But maybe not write yet another CKEditor builder because their builder is crazy and changes...

Gauthier

@gotcha
Copy link
Member Author

gotcha commented Sep 26, 2015

Once again, if you check my code, I am not building a new ckbuilder.

@gbastien
Copy link
Member

I do not know what is the best solution, we just need regular updates and release I think.

Anyway, if tests with version 4.5.3 are ok, we can just forget about 4.4.8 also...

See you,

Gauthier

- scayt plugin was missing
- widget and lineutils dependencies of image2 were also missing
- make script more robusts
@gotcha
Copy link
Member Author

gotcha commented Nov 13, 2015

@gbastien Can you check if c.ckeditor works for you now ?

scayt needs to be enabled by installing the plugin. Was it the case previously ?

@gbastien
Copy link
Member

Hi @gotcha

what do you mean by "installing the plugin"? Scayt is available if you download the full version of ckeditor and will be activated together with ckeditor and other plugins. Isn't it the case anymore?

+++

Gauthier

@gotcha
Copy link
Member Author

gotcha commented Nov 13, 2015

I am asking if installing collective.ckeditor was enough for scayt to be enabled. Or if you had anything else to do ?

@gotcha
Copy link
Member Author

gotcha commented Nov 13, 2015

@gbastien Ok, scayt plugin used to be loaded without any config change. I have fixed the build process.

@gotcha
Copy link
Member Author

gotcha commented Dec 14, 2015

Merged with #46

@gotcha gotcha closed this Dec 14, 2015
@gotcha gotcha deleted the builder branch March 11, 2016 15:11
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.

None yet

2 participants