Skip to content

Conversation

@gumaerc
Copy link
Contributor

@gumaerc gumaerc commented Sep 13, 2024

What are the relevant tickets?

Closes https://github.com/mitodl/hq/issues/5393

Description (What does it do?)

This PR adds a new boolean setting to the app, MITOL_NOINDEX. This setting is meant to control the noindex meta tag in the head section of every page. It is true by default, preventing indexing of pages by search engines like Google. In production, we will set this value to false so that the tag is not written in and pages are indexed as expected.

How can this be tested?

  • In your environment, set MITOL_NOINDEX=true (or omit the value, it defaults to true)
  • Spin up this branch of mit-learn
  • Visit the home page at http://localhost:8062/
  • Inspect the page source and ensure that you see a <meta name="robots" content="noindex" /> had in the head section
  • Change the value to false and reload your containers
  • Visit the homepage and inspect the page source again, verifying that the tag is no longer present

Checklist:

@gumaerc gumaerc added the Needs Review An open Pull Request that is ready for review label Sep 13, 2024
@abeglova abeglova self-assigned this Sep 16, 2024
Copy link
Contributor

@abeglova abeglova left a comment

Choose a reason for hiding this comment

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

👍

@gumaerc gumaerc merged commit 89e9678 into main Sep 16, 2024
@rhysyngsun rhysyngsun deleted the cg/noindex branch February 7, 2025 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review An open Pull Request that is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants