Skip to content

Upgrade CKEditor to 47 using sulu/sulu PR branch#326

Open
alexander-schranz wants to merge 2 commits intosulu:2.6from
alexander-schranz:codex/ckeditor-47-upgrade
Open

Upgrade CKEditor to 47 using sulu/sulu PR branch#326
alexander-schranz wants to merge 2 commits intosulu:2.6from
alexander-schranz:codex/ckeditor-47-upgrade

Conversation

@alexander-schranz
Copy link
Member

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets n/a
Related issues/PRs sulu/sulu#8652
License MIT
Documentation PR n/a

What's in this PR?

  • Adds a VCS repository override for alexander-schranz/sulu.
  • Uses sulu/sulu branch dev-codex/ckeditor-47-upgrade as 2.6.20.
  • Upgrades admin CKEditor build dependencies in assets/admin/package.json to ^47.0.0.
  • Regenerates admin build assets in public/build/admin.

Why?

Align the skeleton with the CKEditor 47 upgrade from sulu/sulu#8652 and keep the admin build compatible with that Sulu branch.

BC Breaks/Deprecations

None.

To Do

  • Create a documentation PR
  • Create PR for sulu/sulu test skeleton

Comment on lines +29 to +34
"repositories": [
{
"type": "vcs",
"url": "https://github.com/alexander-schranz/sulu"
}
],
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ToDo revert

Suggested change
"repositories": [
{
"type": "vcs",
"url": "https://github.com/alexander-schranz/sulu"
}
],

@alexander-schranz alexander-schranz force-pushed the codex/ckeditor-47-upgrade branch from 91bce36 to 2e820d8 Compare February 24, 2026 12:56
"scheb/2fa-trusted-device": "^7.4",
"stof/doctrine-extensions-bundle": "^1.11",
"sulu/sulu": "~2.6.21",
"sulu/sulu": "dev-codex/ckeditor-47-upgrade as 2.6.21",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

revert:

Suggested change
"sulu/sulu": "dev-codex/ckeditor-47-upgrade as 2.6.21",
"sulu/sulu": "~2.6.21",

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant