Skip to content

[MOOSE-384] FE: Accordion Block Schema - #381

Open
GeoffDusome wants to merge 2 commits into
mainfrom
feature/MOOSE-384/accordion-schema
Open

[MOOSE-384] FE: Accordion Block Schema#381
GeoffDusome wants to merge 2 commits into
mainfrom
feature/MOOSE-384/accordion-schema

Conversation

@GeoffDusome

Copy link
Copy Markdown
Contributor

What does this do/fix?

This pull request introduces support for generating FAQPage structured data (schema.org) for the core/accordion block when a new "Enable FAQ Schema" toggle is enabled. The main changes add both the backend filter to generate the JSON-LD schema and the editor control for users to opt in.

FAQ Schema Support for Accordion Block:

  • Added a new Accordion_Filter class that appends FAQPage structured data to the core/accordion block if the "Enable FAQ Schema" toggle is enabled. The filter parses the block's HTML to extract question and answer pairs and outputs the appropriate JSON-LD.
  • Registered the new Accordion_Filter in the block filters definition so it is applied when rendering blocks. [1] [2]

Block Editor Enhancements:

  • Added a new toggle control in the core/accordion block editor (editor.js) allowing users to enable or disable FAQ schema generation for each accordion instance.

QA

Links to relevant issues

Screenshots/video:

Pull request checklist

  • I've added a changelog entry for these changes.
  • I've linked to a relevant Jira issue.
  • I've captured a screenshot or screencast of the changes and linked it above.

@dpellenwood dpellenwood left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🍏 Nice use fo the Core HTML parser!

@GeoffDusome GeoffDusome added In QA Waiting on QA and removed Needs FE Review labels Aug 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

In QA Waiting on QA Launch Environment Deploys to dokku

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants