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.

Sign up for GitHub

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

Jump to bottom

use existing filter to remove bgseo robots output #52

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

use existing filter to remove bgseo robots output #52

wants to merge 1 commit into from

Commits on Jan 13, 2020

  1. use existing filter to remove bgseo robots output 

    see comment on [BoldGrid/boldgrid-seo#19](BoldGrid/boldgrid-seo#19) for reference.  

This uses the preexisting filter in bgseo to remove the robots output instead of adding a new filter for this one off case.  So the superfluous addition of `boldgrid-seo/seo/robots/run` can be reverted in bgseo repo which will help keep the existing filters on output methods unified in that plugin.  There have been a few support tickets where filters in boldgrid-seo were modified, and the [BoldGrid/boldgrid-seo-anspress-extension](https://github.com/BoldGrid/boldgrid-seo-anspress-extension) makes use of the filters as an officially released plugin that extends boldgrid-seo.

`boldgrid-seo/seo/robots/run` was added to inspirations when implementing #49 for further reference if needed.
    @timelsass
    timelsass authored Jan 13, 2020
    Configuration menu
    Copy the full SHA
    8fa377e View commit details
    Browse the repository at this point in the history