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

Change global variable names #16736

Closed
filipsobol opened this issue Jul 16, 2024 · 0 comments
Closed

Change global variable names #16736

filipsobol opened this issue Jul 16, 2024 · 0 comments
Labels
resolution:resolved This issue was already resolved (e.g. by another ticket). squad:core Issue to be handled by the Core team. type:improvement This issue reports a possible enhancement of an existing feature.

Comments

@filipsobol
Copy link
Member

filipsobol commented Jul 16, 2024

In the release 42.0.0, alongside the ESM browser builds we also introduced UMD browser builds. The latter build was not mentioned in the changelog nor the documentation because we weren't ready to promote this installation methods yet (and they shouldn't be used yet).

After the discussion, we decided that we're not happy with the names of the global variables we used that match the names of the npm packages (ckeditor5 and ckeditor5-premium-features) and instead we should use the following names:

@filipsobol filipsobol added the type:improvement This issue reports a possible enhancement of an existing feature. label Jul 16, 2024
pomek added a commit to ckeditor/ckeditor5-react that referenced this issue Jul 17, 2024
Fix: Change the global name used in the UMD build from `CKEditor` to `CKEDITOR_REACT` to match the new convention. See ckeditor/ckeditor5#16736.

MINOR BREAKING CHANGE: Change the global name used in the UMD build from `CKEditor` to `CKEDITOR_REACT`.
pomek added a commit to ckeditor/ckeditor5-vue that referenced this issue Jul 17, 2024
Fix: Change the global name used in the UMD build from `CKEditor` to `CKEDITOR_VUE` to match the new convention. See ckeditor/ckeditor5#16736.

MINOR BREAKING CHANGE: Change the global name used in the UMD build from `CKEditor` to `CKEDITOR_VUE`.
@Witoso Witoso added squad:core Issue to be handled by the Core team. resolution:resolved This issue was already resolved (e.g. by another ticket). labels Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
resolution:resolved This issue was already resolved (e.g. by another ticket). squad:core Issue to be handled by the Core team. type:improvement This issue reports a possible enhancement of an existing feature.
Projects
None yet
Development

No branches or pull requests

2 participants