liferay / liferay-ckeditor

Other
8 stars 49 forks source link

Update to v4.21.0 #206

Closed fortunatomaldonado closed 1 year ago

fortunatomaldonado commented 1 year ago

https://github.com/liferay/liferay-ckeditor/issues/205

Update to CKEditor 4.21.0.

Please let me know if this is the correct process.

Thank you!

markocikos commented 1 year ago

@fortunatomaldonado When you updated base version, were our 15 commits successfully rebased on top of liferay branch on ckeeditor-dev? If there were no conflicts, great!

What's missing in this PR is the update to patches folder, probably you just need to do ./ck.sh patch. Add that commit, and we can merge PR. I believe this does not affect build artifacts, that our changes were included.

You also need to publish new version on npm. There should be a new 4.21.0-liferay.1 version here.

fortunatomaldonado commented 1 year ago

Hello @markocikos, I did find a conflict with patch 0013 that I was able to fix and merge. I then updated the patches. Let me know if there is anything else needed to be fixed.

Once this is merge, I should create the new version on npm correct?

Thank you for taking a look at this!

markocikos commented 1 year ago

@fortunatomaldonado are you sure that changes in patches are included in the latest build artifacts?

fortunatomaldonado commented 1 year ago

@markocikos, I believe so but I could be wrong. I went into ckeditor-web, made the changes, ran git add and git am --continue and then ran build.

Please correct me if I am messing up the process. Thank you for your help!

fortunatomaldonado commented 1 year ago

Hello @markocikos, Is there some other way you would like me to try this? Thank you!

markocikos commented 1 year ago

@fortunatomaldonado just doublechecking, but this looks good! After this is merged, you can do the publish part. Note that the publish script both pushes to this repo and to npm, hopefully all permissions are set for you. The version we want is 4.21.0-liferay.1. If something goes wrong, let me know and I'll complete the release.