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 wrong sitemap for blog #801

Merged
merged 1 commit into from
Jun 26, 2018
Merged

Fix wrong sitemap for blog #801

merged 1 commit into from
Jun 26, 2018

Conversation

endiliey
Copy link
Contributor

Motivation

#799 (comment)

Our sitemap url for blog is always wrong due to wrong logic.

Consider this
consider

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

Before
before

After
after

Related PRs

#765

@endiliey endiliey requested a review from JoelMarcey June 25, 2018 05:00
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Jun 25, 2018
@endiliey endiliey requested a review from yangshun June 25, 2018 05:01
@docusaurus-bot
Copy link
Contributor

Deploy preview for docusaurus-preview ready!

Built with commit 2c753b4

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

@endiliey endiliey added bug An error in the Docusaurus core causing instability or issues with its execution ✅ Test Plan labels Jun 25, 2018
@JoelMarcey
Copy link
Contributor

I think that one may have been my fault 🤔

😂

Approved

Sent with GitHawk

@endiliey endiliey merged commit 418c840 into facebook:master Jun 26, 2018
@endiliey endiliey deleted the sitemap branch June 26, 2018 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An error in the Docusaurus core causing instability or issues with its execution CLA Signed Signed Facebook CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants