Skip to content

Conversation

@Fran-A-Dev
Copy link
Contributor

I added a tip to users when reading it to check their WP sitemap domain.

@headless-platform-by-wp-engine

Check out the recent updates to your Headless Platform preview environment:

App Environment URL Build
faustjs.org preview-env-fix/sitemap-doc-note https://hp…wered.com ✅ (logs)

Learn more about preview environments in our documentation.

@colinmurphy
Copy link
Member

LGTM 🎉 Thanks @Fran-A-Dev 😄


![WordPress Reading Settings screen displaying options for feed content inclusion and search engine visibility. The 'Discourage search engines from indexing this site' checkbox is unchecked, ensuring the site is visible to search engines.](./images/reading-settings.png)

> [!TIP]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why would a user want to do this? Is this a step they should do or something they do only if they see other errors?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is a must- do. The the sitemap path breaks and 404's if that box is checked.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Callouts are generally for accessory information. TIPs, in particular, should be read as optional. This sounds like it should be its own step.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call, I'll change it to its own step.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@moonmeister this is good to merge now, committed and pushed the change

@moonmeister
Copy link
Member

@Fran-A-Dev I'm not seeing that change yet, did you forget to push code?

@Fran-A-Dev
Copy link
Contributor Author

@Fran-A-Dev I'm not seeing that change yet, did you forget to push code?

Oh word, I thought I pushed it. I think I just committed it locally. Let me push

@headless-platform-by-wp-engine

Check out the recent updates to your Headless Platform preview environment:

App Environment URL Build
faustjs.org preview-env-fix/sitemap-doc-note https://hp…wered.com ✅ (logs)

Learn more about preview environments in our documentation.

@moonmeister moonmeister merged commit 97f26cc into toolkit Feb 10, 2025
1 check passed
@moonmeister moonmeister deleted the fix/sitemap-doc-note branch February 10, 2025 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants