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

Feature/remove 10up author pages #43

Merged
merged 6 commits into from
Feb 5, 2019

Conversation

bengreeley
Copy link
Contributor

Redirects author pages for any users that have an @10up.com or @get10up.com e-mail address on a production website.

helen and others added 3 commits November 4, 2018 17:09
…for any users with @get10up.com or @10up.com e-mail address in order to prevent Google from archiving users who may have created pages but dont contribute to the site.
@tlovett1
Copy link
Member

tlovett1 commented Feb 5, 2019

Thanks @bengreeley !

@tlovett1 tlovett1 merged commit fa8aa5d into 10up:develop Feb 5, 2019
@tlovett1
Copy link
Member

tlovett1 commented Feb 5, 2019

@bengreeley thinking about this a bit... is 301 redirect the right move versus a 404? Should we worry about sitemaps?

@bengreeley
Copy link
Contributor Author

@tlovett1 I think 301 would be fine since it would be more graceful for search engines to have a 301 to indicate that they should update their records as opposed to a hard 404. As far as sitemaps, I think that's an interesting consideration. I would think we'd need to plan for specific sitemap plugins for Yoast or msm-sitemap since there aren't standard sitemap generation functions that both use. I'll see if I can think through a solution for that which would be more generic.

@bengreeley bengreeley deleted the feature/remove-10up-author-pages branch March 8, 2019 19:22
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.

None yet

3 participants