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 SEO: Update Configuration section to improve search #24759

Merged
merged 6 commits into from
Jan 10, 2025
Merged

Docs SEO: Update Configuration section to improve search #24759

merged 6 commits into from
Jan 10, 2025

Conversation

aimeeu
Copy link
Contributor

@aimeeu aimeeu commented Dec 27, 2024

Description

  • Front matter description updates for DevDot search.
  • Add intro body paragraph that summarizes page content ("This page..."). This is basically the same verbiage as the front matter description but in the body for better Google search indexing.
  • Ensure front matter page_title and # <title> match.

I mostly did not edit page content because that's out of scope for this effort. I did some small replacements tho ("see", "below"). Added a few links to pages that didn't have any (orphan pages).

A member of the Runtime Education team will review all SEO PRs.

Links

Jira: CE-775
Preview: https://nomad-git-ce775-hashicorp.vercel.app/nomad/docs/configuration

Contributor Checklist

  • Changelog Entry If this PR changes user-facing behavior, please generate and add a
    changelog entry using the make cl command.
  • Testing Please add tests to cover any new functionality or to demonstrate bug fixes and
    ensure regressions will be caught.
  • Documentation If the change impacts user-facing functionality such as the CLI, API, UI,
    and job configuration, please update the Nomad website documentation to reflect this. Refer to
    the website README for docs guidelines. Please also consider whether the
    change requires notes within the upgrade guide.

Reviewer Checklist

  • Backport Labels Please add the correct backport labels as described by the internal
    backporting document.
  • Commit Type Ensure the correct merge method is selected which should be "squash and merge"
    in the majority of situations. The main exceptions are long-lived feature branches or merges where
    history should be preserved.
  • Enterprise PRs If this is an enterprise only PR, please add any required changelog entry
    within the public repository.

@aimeeu aimeeu requested review from a team as code owners December 27, 2024 19:21
@aimeeu aimeeu requested review from pbortnick and removed request for pbortnick December 27, 2024 19:21
@aimeeu aimeeu added theme/docs Documentation issues and enhancements backport/website This will backport PR changes to `stable-website` && the latest release-branch backport/1.9.x backport to 1.9.x release line labels Dec 27, 2024
philrenaud
philrenaud previously approved these changes Dec 30, 2024
Copy link
Contributor

@philrenaud philrenaud left a comment

Choose a reason for hiding this comment

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

Oh, neat. Does this mean this'll be more likely to come up for searches beginning with "How do I...." given the "Use... to..." answers?

LGTM!

@aimeeu
Copy link
Contributor Author

aimeeu commented Jan 2, 2025

Oh, neat. Does this mean this'll be more likely to come up for searches beginning with "How do I...." given the "Use... to..." answers?

LGTM!

I'm following description templates created by the Education team so only time will tell :)

Copy link
Contributor

@boruszak boruszak left a comment

Choose a reason for hiding this comment

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

I left three suggestions on the first three descriptions with longer comments that explain the kinds of things we should be thinking about when updating descriptions for SEO. Your current edits are more focused on the style guide & active voice. But active voice isn't going to impact SEO.

This page from Google's SEO docs is their summary. The use quality descriptions subsection may be a useful resource.

website/content/docs/configuration/acl.mdx Outdated Show resolved Hide resolved
website/content/docs/configuration/audit.mdx Outdated Show resolved Hide resolved
website/content/docs/configuration/autopilot.mdx Outdated Show resolved Hide resolved
@aimeeu aimeeu changed the title Docs SEO: Update Configuration section to improve search engine optimization Docs SEO: Update Configuration section to improve search engine optimization DRAFT Jan 7, 2025
@aimeeu aimeeu marked this pull request as draft January 7, 2025 14:22
@aimeeu aimeeu changed the title Docs SEO: Update Configuration section to improve search engine optimization DRAFT Docs SEO: Update Configuration section to improve search Jan 9, 2025
boruszak
boruszak previously approved these changes Jan 9, 2025
Copy link
Contributor

@boruszak boruszak left a comment

Choose a reason for hiding this comment

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

Amazing work rewriting these descriptions.

My comments are mostly teeny nitpicky things. Overall, you've done a tremendous job on this set of descriptions.

website/content/docs/configuration/index.mdx Outdated Show resolved Hide resolved
website/content/docs/configuration/keyring/transit.mdx Outdated Show resolved Hide resolved
website/content/docs/configuration/keyring/transit.mdx Outdated Show resolved Hide resolved
website/content/docs/configuration/reporting.mdx Outdated Show resolved Hide resolved
website/content/docs/configuration/search.mdx Outdated Show resolved Hide resolved
website/content/docs/configuration/search.mdx Outdated Show resolved Hide resolved
@aimeeu aimeeu merged commit ffb3431 into main Jan 10, 2025
16 checks passed
@aimeeu aimeeu deleted the ce775 branch January 10, 2025 17:05
aimeeu added a commit that referenced this pull request Jan 10, 2025
* Docs SEO: Update Configuration section to improve search engine opt

CE-775

* Add enterprise only back to audit

* Update descriptions and add intro paragraph

* Fix typo

* replace "below" and "see"

* Apply suggestions from code review

Co-authored-by: Jeff Boruszak <[email protected]>

---------

Co-authored-by: Jeff Boruszak <[email protected]>
aimeeu added a commit that referenced this pull request Jan 10, 2025
…4835)

* Docs SEO: Update Configuration section to improve search engine opt

CE-775

* Add enterprise only back to audit

* Update descriptions and add intro paragraph

* Fix typo

* replace "below" and "see"

* Apply suggestions from code review



---------

Co-authored-by: Jeff Boruszak <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/website This will backport PR changes to `stable-website` && the latest release-branch backport/1.9.x backport to 1.9.x release line theme/docs Documentation issues and enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants