Non-RSS page called feeds omitted in generated output

Hey @tremlas,

we indeed added “feed” as a keyword to the excluded list for URLs unless the add feeds option is enabled:

// Exclude feeds if add_feeds is not true.
if ( ! $this->options->get( ‘add_feeds’ ) ) {
$excluded[] = ‘feed’;
}

I have to do some testing, to see if it’s still happening if the option is enabled, but you can also remove it yourself with the following filter:

add_filter(‘ss_excluded_by_default’, function ($excluded) {
$excluded[] = ‘wp-json’;
$excluded[] = ‘.php’;
$excluded[] = ‘debug’;
return $excluded;
});

I tried a generation with your suggested filter at the end of my theme’s functions.php but that didn’t change the result (the /feeds/index.html file is still not present in the resulting zip).

I also took a look at earlier generations. One from 25th Dec was as I expected. One from 19th Jan (which would have used the 3.2.5.3 version of simply static) gained a /feed/index.html as well as the /feeds/index.html – this contains a redirect to index.xml). Today’s generation contains neither file (to clarify that feed/index.html redirect is not one I’ve created but presumably a standard one from wordpress itself).

I’m guessing there’s an interaction between the change that (presumably) accidently added the /feed url and the one that removes it.

Hey @tremlas found the problem and fixed it now: https://github.com/Simply-Static/simply-static/commit/2683d7f53759f4218f9a4ed11a42f004daa44fd2

The filter was implemented too early, so while the code snippet I recommended is the right approach to handling it, it was simply overwritten by the new feed addition. The next update will handle it appropriately.