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

docs(v2): Reorganize migration guide #2036

Merged
merged 4 commits into from
Nov 25, 2019

Conversation

wgao19
Copy link
Contributor

@wgao19 wgao19 commented Nov 23, 2019

Motivation

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

NA

Related PRs

NA

@docusaurus-bot
Copy link
Contributor

docusaurus-bot commented Nov 23, 2019

Deploy preview for docusaurus-2 ready!

Built with commit 05e3cd7

https://deploy-preview-2036--docusaurus-2.netlify.com

@docusaurus-bot
Copy link
Contributor

docusaurus-bot commented Nov 23, 2019

Deploy preview for docusaurus-preview ready!

Built with commit 05e3cd7

https://deploy-preview-2036--docusaurus-preview.netlify.com

@endiliey
Copy link
Contributor

Thank you :) let me know when its no longer WIP

@wgao19
Copy link
Contributor Author

wgao19 commented Nov 23, 2019

@endiliey I'm not very familiar with remark-toc, do you mind adding the guide here?

@endiliey endiliey added the pr: documentation This PR works on the website or other text documents in the repo. label Nov 23, 2019
@endiliey
Copy link
Contributor

I think just writing

it has been deprecated, you might want to use remark-toc instead and pass it to docs plugin remarkPlugins option

@wgao19
Copy link
Contributor Author

wgao19 commented Nov 23, 2019

what really, dont we want to include how to pass that...

@endiliey
Copy link
Contributor

Actually using remark-toc isnt really a direct workaround. Its a different syntax. Just point that it has been deprecated, and link to that github issue instead for reasoning.

@wgao19 wgao19 changed the title WiP: Reorganize migration guide Reorganize migration guide Nov 25, 2019
@wgao19 wgao19 requested a review from endiliey November 25, 2019 12:27
@endiliey endiliey changed the title Reorganize migration guide docs(v2): Reorganize migration guide Nov 25, 2019
@endiliey endiliey merged commit 2254e08 into master Nov 25, 2019
@endiliey endiliey deleted the wgao19/docs/reorganize-migration-guide branch November 25, 2019 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA pr: documentation This PR works on the website or other text documents in the repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants