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/keyword analysis #18

Merged
merged 9 commits into from
Mar 7, 2021
Merged

Feature/keyword analysis #18

merged 9 commits into from
Mar 7, 2021

Conversation

schultyy
Copy link
Owner

@schultyy schultyy commented Mar 7, 2021

This PR ships with more sophisticated keyword analysis for frontmatter and the article header.

  • If the user has several keywords configured, only the top two keyword should appear in seo_title.
  • If the user has several keywords configured, only the top one should appear in the article title.
  • If the user has several keywords configured, only the top two should appear in seo_description.
  • Makes sure the top keyword doesn't appear more than once in seo_description.
  • We partially match the keyword in seo_title and the article title.

Resolves #5

@schultyy schultyy marked this pull request as ready for review March 7, 2021 18:07
@schultyy schultyy merged commit b09ff39 into main Mar 7, 2021
@schultyy schultyy deleted the feature/keyword-analysis branch March 7, 2021 18:08
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.

Keyword ranking
1 participant