Skip to content
This repository has been archived by the owner on Mar 16, 2020. It is now read-only.

Add a sitemap.txt with all courses #201

Merged
merged 1 commit into from
Oct 5, 2018
Merged

Add a sitemap.txt with all courses #201

merged 1 commit into from
Oct 5, 2018

Conversation

fofr
Copy link
Contributor

@fofr fofr commented Oct 4, 2018

Use basic text list of URLs. We will need to eventually autogenerate this somehow.

https://support.google.com/webmasters/answer/183668?hl=en&ref_topic=4581190

Use basic text list of URLs. We will need to eventually autogenerate this somehow.

https://support.google.com/webmasters/answer/183668?hl=en&ref_topic=4581190
@fofr fofr requested a review from benilovj October 4, 2018 16:00
Copy link
Contributor

@defong defong left a comment

Choose a reason for hiding this comment

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

looks fine to me

@benilovj
Copy link
Contributor

benilovj commented Oct 5, 2018

Not sure about this being static.

@fofr
Copy link
Contributor Author

fofr commented Oct 5, 2018

@benilovj Aye, I agree. This is MVP.

@fofr
Copy link
Contributor Author

fofr commented Oct 5, 2018

@fofr fofr merged commit 5a80606 into master Oct 5, 2018
@fofr fofr deleted the sitemap branch October 5, 2018 10:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants