Skip to content

fixed(markdown): render the markdown in browser. - #505

Closed
ghost wants to merge 1 commit into
masterfrom
unknown repository
Closed

fixed(markdown): render the markdown in browser.#505
ghost wants to merge 1 commit into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Apr 15, 2018

Copy link
Copy Markdown

fixes #451
closes #451

@lvxianchao

Copy link
Copy Markdown

Is this repository died? I found this bug today.

Comment thread lib/Markdown.js
*/
render(options, cb) {
return this._request('POST', '/markdown', options, cb);
return this._request('POST', '/markdown', options, cb, true);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm trying to figure out why sending the request as raw rather than JSON would work in the browser. Give me a bit to sort this out.

j-rewerts added a commit to j-rewerts/github that referenced this pull request Apr 7, 2019
@j-rewerts j-rewerts mentioned this pull request Apr 7, 2019
@j-rewerts

Copy link
Copy Markdown
Member

I've created a PR that contains the previous code, as well as a test that verifies this is an issue.

@j-rewerts

Copy link
Copy Markdown
Member

This can be closed @clayreimann.

@clayreimann

Copy link
Copy Markdown
Member

Closing in favor of #548

j-rewerts added a commit to j-rewerts/github that referenced this pull request Apr 10, 2019
clayreimann pushed a commit that referenced this pull request Apr 17, 2019
Got the code from PR #505.
Fixes #451.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

How can I render the markdown in browser,may be a bug

3 participants