withspectrum/draft-js-markdown-plugin

DraftJS v0.11.0 breaks plugin

d3jawu opened this issue · 4 comments

d3jawu commented

When pairing the plugin with DraftJS v0.11.0, the console will give the error setEditorState is not a function anytime a Markdown shortcut is used.

Looks like there was an API change that may have affected this - I'm not familiar enough yet with DraftJS to know.

Using DraftJS v0.10.4 allows the plugin to work properly.

elbakerino commented

Just as an information, saw this issue during debugging.

I got the base repo runnning with 0.11 and opened a PR there. Maybe the change helps in this fork also.

Arthur944 commented

I have the same issue too

tom-and-jary commented

I have the same issue too

thecodedrift commented

EDIT PR available at #186

There should probably be a peerDependency on DraftJS at ^0.10.5 similar to how we specify the plugin editor versions that are supported. However, fixing this (and supporting the new parameter in handleBeforeInput) will cause us to rev our peerDependency list; that'd be a bump to major and require folks to upgrade both DraftJS and the Plugin Wapper.

If you're blocked by this, make sure you're using draft-js @ ^0.10.5 and draft-js-plugins-editor @ ^2.1.1.

I'm all for 4.x of this plugin, but it's a small change with some pretty major implications. In the meanwhile, I'll put together a PR here on this fork that includes @elbakerino's changes + updates the peerDependencies. It will push us to 4.0.0 since these changes will be backwards incompatible with draft 0.10 and the 2.x branch of the plugins editor.

@mxstbr could you weigh on on this?