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

Remove the need for seoContentFields #16

Open
zanderwar opened this issue Apr 18, 2018 · 1 comment
Open

Remove the need for seoContentFields #16

zanderwar opened this issue Apr 18, 2018 · 1 comment
Labels
enhancement New feature or request

Comments

@zanderwar
Copy link
Contributor

Remove the need for seoContentFields() by using the DOM parser to better analyse the page content.

We have to consider that majority of the time there is always duplicate content across the site, which would be the header, nav and footer. We can assume that these tags are used correctly and great content is not placed within them, and then remove these entirely before the analysis.

I mean to get anymore accurate than that you would need cross-page analysis to determine the probability of what is duplicate content and what is not.

Currently, seoContentFields does not support many_many or has_many relations so may very-greatly reduce the accuracy of the current analysis.

@chillu
Copy link

chillu commented Jul 16, 2019

Note that there's a related core RFC for this now: silverstripe/silverstripe-cms#2454

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants