Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

T/22: Move image upload feature to ckeditor5-image package. #81

Merged
merged 11 commits into from
Feb 2, 2018
Merged

Conversation

jodator
Copy link
Contributor

@jodator jodator commented Feb 1, 2018

Suggested merge commit message (convention)

Other: Removed image upload feature. Closes ckeditor/ckeditor5#2800.

BREAKING CHANGE: Renamed Adapter to UploadAdapter.
BREAKING CHANGE: Removed ImageUpload plugin. It can be no found in ckeditor5-image repository.
BREAKING CHANGE: Removed ImageUploadEngine plugin. It can be no found in ckeditor5-image repository.
BREAKING CHANGE: Removed ImageUploadProgress plugin. It can be no found in ckeditor5-image repository.
BREAKING CHANGE: Removed ImageUploadButton plugin. It can be no found in ckeditor5-image repository.
BREAKING CHANGE: Renamed FileRepository#createAdapter() to FileRepository#createUploadAdapter().
BREAKING CHANGE: Renamed filerepository-no-adapter error to filerepository-no-upload-adapter.


Additional information

This PR required tests and docs changes in other repos (centralized in: https://github.com/ckeditor/ckeditor5/tree/t/ckeditor5-upload/22) - the main repo also needs changes without last commit.

The image PR has all the changes in the API:

Merge commit message for related branches:

Internal: Aligned code to the changes in the file upload API. (see ckeditor/ckeditor5#2800).

@coveralls
Copy link

coveralls commented Feb 1, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling bd77c74 on t/22 into 9cd8053 on master.

@jodator jodator requested a review from Reinmar February 1, 2018 10:52
@Reinmar Reinmar merged commit bd77c74 into master Feb 2, 2018
@Reinmar Reinmar deleted the t/22 branch February 2, 2018 10:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Review contents of this package
3 participants