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

Parse the text value in code blocks #152

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

stinoga
Copy link

@stinoga stinoga commented Nov 8, 2018

This resolves an issue where code block child nodes are returning
strings only, rather than a TextNode. This causes them to fail to
render when imported.

This resolves an issue where code block child nodes are returning
strings only, rather than a TextNode. This causes them to fail to
render when imported.
@sstur
Copy link
Owner

sstur commented Jan 3, 2019

Hey @stinoga, thanks for your contribution!

Can you add a test case to this file, so I can better understand what issue this solves?

https://github.com/sstur/draft-js-utils/blob/master/packages/draft-js-import-markdown/src/__tests__/stateFromMarkdown-test.js

Ideally some piece of markdown that is incorrectly parsing with the current implementation and fixed with your PR.

Thanks in advance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants