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

Clean-up model, document and controllers API #1215

Merged
merged 4 commits into from
Dec 26, 2017
Merged

Clean-up model, document and controllers API #1215

merged 4 commits into from
Dec 26, 2017

Conversation

ma2ciek
Copy link
Contributor

@ma2ciek ma2ciek commented Dec 21, 2017

Suggested merge commit message (convention)

Other: Cleaned up model, document and controllers API. Closes ckeditor/ckeditor5#4220.


Additional information

BREAKING CHANGE: Moved tranformation imports from Document to Model
BREAKING CHANGE: Moved document.transformDeltas() to model.transformDeltas()
BREAKING CHANGE: Moved dataController.insertContent() to model.insertContent()
BREAKING CHANGE: Moved dataController.deleteContent() to model.deleteContent()()
BREAKING CHANGE: Moved dataController.modifySelection to model.modifySelection()
BREAKING CHANGE: Moved dataController.getSelectedContent to model.getSelectedContent()
BREAKING CHANGE: Moved dataController.hasContent to model.hasContent()
BREAKING CHANGE: Moved utils from controller to model/utils

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 2ef1319 on t/1208 into ** on master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling f864e43 on t/1208 into ** on master**.

@pjasiun
Copy link

pjasiun commented Dec 22, 2017

The change looks fine. However Friday before Christmas, 17:00 sounds like a terrible moment for a multirepo merge, so let's wait a moment :)

@pjasiun pjasiun self-requested a review December 22, 2017 16:08
@Reinmar Reinmar merged commit aea6119 into master Dec 26, 2017
@Reinmar Reinmar deleted the t/1208 branch December 26, 2017 20:02
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.

Clean-up model, document and controllers API
4 participants