Allow configuration of trailing slashes for meta and sitemap

[icemancast]

A client needs the sitemap urls to have the appended slash in the url. How can I achieve this?

[jesseleite]

This is probably something that should be configurable in Statamic core, so that packages like seo-pro and ssg can opt-into trailing slashes. We've thought about an implementation on the URL facade, maybe something like URL::enforceTrailingSlashes(bool), so that it can be toggled on and off for specific situations without affecting the whole app.

Related statamic/ideas#1293
Related statamic/ssg#65

[jesseleite]

@icemancast In the meantime, you can extend the Sitemap class and override the pages() method by following the steps in this PR to rebind your custom sitemap class extension. Something like this will forcibly append trailing slashes to all your sitemap URLs...

class CustomSitemap extends Sitemap
{
    public function pages(): array
    {
        return collect(parent::pages())
            ->map(fn ($page) => array_merge($page, [
                'loc' => $page['loc'].'/',
            ]))
            ->all();
    }
}

[icemancast]

Awesome thanks @jesseleite , google seems to be adding rules by the minute for everything and marketers seem to keep hitting me with something new I need to help with.

[jesseleite]

@icemancast For sure!

Fwiw, I'm not convinced trailing slashes matter. I think this is often a misunderstood SEO requirement. For example, looking at Google's guide, which references sitemaps.org, where the protocol only requires trailing slashes if your web server requires them to serve the page...

That said, I still don't think it hurts to make it configurable! ❤️

[icemancast]

@jesseleite thank you this helps my case too :).

[icemancast]

@jesseleite I did figure out the issue. So this statamic sites shoots to netlify and it compiles the static files. Netlify by default appends the forward slash which doesn't match the sitemap. So their seo program is complaining about the issue.

[jesseleite]

@icemancast Ah good to know, that makes sense!