Skip to content

Comments

Documentation for the schema aggregator#378

Open
thijsoo wants to merge 5 commits intomainfrom
schema-aggregation-docs
Open

Documentation for the schema aggregator#378
thijsoo wants to merge 5 commits intomainfrom
schema-aggregation-docs

Conversation

@thijsoo
Copy link
Contributor

@thijsoo thijsoo commented Feb 2, 2026

Fixes #

Summary

Relevant technical choices

Technical choices that affect more than this issue:

Test instructions

This PR can be acceptance tested by following these steps:
1.
1.
1.

Quality assurance

  • Security - I have thought about any security implications this code might add.
  • Performance - I have checked that this code doesn't impact performance (greatly).
  • Caching - I have analyzed the caching methods that this code touches and have added instructions to deal with those.
  • Tested - I have tested this code to the best of my abilities.
  • Automated tests - I have added unit tests to verify the code works as intended.
  • Testability - I have added unique ids to elements, so they can be located in automated testing.
  • I have altered a filename.
    • I have adjusted the ID property accordingly and updated all internal links.
    • I have added the redirect to the _redirects file in the root of the project.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 2, 2026

Deploying yoast-developer with  Cloudflare Pages  Cloudflare Pages

Latest commit: 603f57b
Status: ✅  Deploy successful!
Preview URL: https://3ebc1067.yoast-developer.pages.dev
Branch Preview URL: https://schema-aggregation-docs.yoast-developer.pages.dev

View logs

@thijsoo thijsoo assigned pls78 and thijsoo and unassigned pls78 Feb 2, 2026
@thijsoo thijsoo assigned pls78 and unassigned thijsoo Feb 5, 2026
@pls78 pls78 assigned thijsoo and unassigned pls78 Feb 24, 2026
Copy link
Contributor Author

@thijsoo thijsoo left a comment

Choose a reason for hiding this comment

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

ACC 👍 Lets merge this when the feature goes live :)

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.

2 participants