Trailing slash support (given it's now default on Gatsby).

emiliekmartinez commented 1 year ago

Welcome to the Gatsby Plugin Advanced Sitemap GitHub repo! 👋🎉

Issue Summary

As of Gatsby gatsby@4.7.0, trailing slashes are now the default. The sitemap should therefore also include these trailing slashes in the unique URLs. I think it'd also make sense to follow the pattern of adding a flag if someone wants to remove the trailing slash.

To Reproduce

Follow the instructions in the ReadMe.

Other info

E.g. all of these should have slash's at the end of them, but they don't.

Technical details:

Gatsby Version: 4.7+
Node Version:
OS:
gatsby-config.js:

{
      resolve: `gatsby-plugin-advanced-sitemap`,
      options: {
          query: `
          {
            allContentfulTwentyTwo {
              edges {
                node {
                  slug
                  id
                  updated_at: updatedAt
                  }
                }
              }
            }
            allContentfulTwentyTwoArticle {
              edges {
                node {
                  slug
                  id
                  updated_at: updatedAt
                  }
                }
              }
            }
          }`,
          mapping: {
            allContentfulTwentyTwoArticle: {
              sitemap: `posts`,
            },
            allContentfulTwentyTwo: {
              sitemap: `pages`,
            },
          },
          exclude: [
            `/dev-404-page`,
            `/404`,
            `/404.html`,
            `/offline-plugin-app-shell-fallback`
          ],
          createLinkInHead: true, // optional: create a link in the `<head>` of your site
          // addUncaughtPages: true, // optional: will fill up pages that are not caught by queries and mapping and list them under `sitemap-pages.xml`
      }
    },