Add Simple XML Sitemap defaults for Event content type

Working on it.

Hi,
Created an MR to add Simple XML Sitemap defaults for Event content type.
It successfully adds 'Events' to be included in /admin/config/search/simplesitemap/entities.

Please review.

This appears to be failing tests for legitimate reasons: https://git.drupalcode.org/issue/drupal_cms-3494066/-/jobs/3740413#L100

The tests appear to be passing now.

Please review.

Assigning to Jim for input.

The sitemap is also part of our SEO offerings, rather than any one content type, so switching track.

While this is an SEO feature, ultimately, it needs to end up in the Events recipe.

Reviewed. Looks okay from a code perspective, but needs a couple of changes. This will also need a change record; see https://git.drupalcode.org/project/drupal_cms/-/wikis/Committer-Info/Cha... for a template of what that should look like/instructions. You can add a change record at https://www.drupal.org/node/add/changenotice?field_project=3466967&field... → .

Looks good to me. Assigning to SEO lead @thejimbirch for sign-off/RTBC.

Looks good, thank you for the contribution!

Marking as RTBC.

Actually, one piece of feedback didn't get resolved here. From #12:

Additionally, any composer.json file that lists the Events recipe as a dependency will need to have its version bumped. In this case, I think that's only project_template/composer.json. So in that file, you'll need to change this line:

"drupal/drupal_cms_events": "^1"

to this:

"drupal/drupal_cms_events": "^1.0.3"

Merged into 1.x and ported to 1.0.x. Thanks all!

After some discussion with @thejimbirch, I have reverted this and am closing it as a duplicate, in favor of the lighter and more complete approach outlined in #3494070-8: Add Simple XML Sitemap defaults for content types, menu and vocabulary → .

I will transfer credit to that issue.