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

Fix Wikipedia link #9429

Closed
wants to merge 1 commit into from
Closed

Fix Wikipedia link #9429

wants to merge 1 commit into from

Conversation

dijonkitchen
Copy link
Contributor

@dijonkitchen dijonkitchen commented Aug 16, 2016

The end of the Wikipedia link has a parenthesis, which was incorrectly used as end of markdown format for link.

The end of the Wikipedia link has a parenthesis, which was incorrectly used as end of markdown format for link.
@ghost
Copy link

ghost commented Aug 16, 2016

By analyzing the blame information on this pull request, we identified @JoelMarcey and @mtunique to be potential reviewers.

@ghost ghost added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Aug 16, 2016
@javache
Copy link
Member

javache commented Aug 16, 2016

@facebook-github-bot shipit

@ghost ghost added GH Review: accepted Import Started This pull request has been imported. This does not imply the PR has been approved. and removed GH Review: review-needed labels Aug 16, 2016
@ghost
Copy link

ghost commented Aug 16, 2016

Thanks for importing.If you are an FB employee go to Phabricator to review internal test results.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 16, 2016
@ghost ghost closed this in 7ca3764 Aug 16, 2016
@ghost ghost added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 16, 2016
@dijonkitchen dijonkitchen deleted the patch-1 branch August 17, 2016 18:09
facebook-github-bot pushed a commit that referenced this pull request Oct 3, 2016
Summary:
The markdown parser (Marked) used by the docs doesn't handle links ending with a closing bracket. The result is a broken link in the [Integration With Existing Apps](http://facebook.github.io/react-native/docs/integration-with-existing-apps.html#our-sample-app) guide.

Using HTML makes the parser ignoring these links.

Note: this is a well-known issue of Marked: markedjs/marked#690
Note bis: The fix #9429 didn't solve the problem.
Closes #10225

Differential Revision: D3964341

fbshipit-source-id: 61564d1cbb632fa1f0c71e7c6cbdc0b9a200caa4
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Import Started This pull request has been imported. This does not imply the PR has been approved.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants