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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validate a need for ckeditor5-essentials to be exposed as DLL-consumer plugin #8576

Closed
jodator opened this issue Dec 3, 2020 · 1 comment
Labels
domain:dx This issue reports a developer experience problem or possible improvement. domain:extending-builds package:essentials resolution:duplicate This issue is a duplicate of another issue and was merged into it. status:discussion type:improvement This issue reports a possible enhancement of an existing feature.

Comments

@jodator
Copy link
Contributor

jodator commented Dec 3, 2020

馃摑 Provide a description of the improvement

During the work on #8517 we've identified that we can't have a DLL that consumes other DLL. For this reason, the editor DLL builds imports plugins required by the Essentials plugin directly.

This is a follow-up to the #8395, but does not block it in any way IMO.


If you'd like to see this improvement implemented, add a 馃憤 reaction to this post.

@jodator jodator added type:improvement This issue reports a possible enhancement of an existing feature. status:discussion package:essentials domain:dx This issue reports a developer experience problem or possible improvement. squad:dx labels Dec 3, 2020
@jodator jodator added this to the iteration 39 milestone Dec 3, 2020
@Reinmar Reinmar modified the milestones: iteration 39, nice-to-have Dec 15, 2020
@Reinmar
Copy link
Member

Reinmar commented Dec 15, 2020

DUP of #8605. We plan to have essentials and other plugins that are included in the existing builds, bundled into the DLL-compatible editor bundles.

@Reinmar Reinmar closed this as completed Dec 15, 2020
@Reinmar Reinmar modified the milestones: nice-to-have, iteration 39 Dec 15, 2020
@pomek pomek added the resolution:duplicate This issue is a duplicate of another issue and was merged into it. label Dec 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:dx This issue reports a developer experience problem or possible improvement. domain:extending-builds package:essentials resolution:duplicate This issue is a duplicate of another issue and was merged into it. status:discussion type:improvement This issue reports a possible enhancement of an existing feature.
Projects
None yet
Development

No branches or pull requests

3 participants