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

Fix: The upcast code was assuming that the editor uses a HTMLDataProc… #6

Closed
wants to merge 6 commits into from

Conversation

fredck
Copy link
Contributor

@fredck fredck commented Dec 16, 2019

…essor, which is not always true.

Suggested merge commit message (convention)

Fix: code blocks upcasting throws when using the markdown DP. Closes ckeditor/ckeditor5#5977.

@fredck
Copy link
Contributor Author

fredck commented Dec 16, 2019

This PR depends on ckeditor/ckeditor5-core#205, therefore the CI failure.

@Reinmar
Copy link
Member

Reinmar commented Dec 17, 2019

R+ for most of the changes. But I'm unsure about the changes in ckeditor5-core.

@Reinmar Reinmar self-assigned this Dec 17, 2019
@fredck
Copy link
Contributor Author

fredck commented Dec 19, 2019

This PR depends on ckeditor/ckeditor5-core#205, therefore the CI failure.

This dependency has been removed. All green now.

@fredck
Copy link
Contributor Author

fredck commented Jan 17, 2020

R+ for most of the changes. But I'm unsure about the changes in ckeditor5-core.

Just to be clear, the previous commit made it not necessary to touch core anymore.

@fredck
Copy link
Contributor Author

fredck commented Feb 25, 2020

Force-pushed a rebase over v17.0.0.

@fredck
Copy link
Contributor Author

fredck commented Mar 21, 2020

Force-pushed a rebase over v18.0.0.

@fredck
Copy link
Contributor Author

fredck commented May 2, 2020

Force-pushed a rebase over v19.0.0.

@fredck
Copy link
Contributor Author

fredck commented May 26, 2020

Moved to ckeditor/ckeditor5#7297.

@fredck fredck closed this May 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants