Adds Schema.org schemas to pages #408

JakubMastalerz · 2025-10-16T13:10:44Z

Link to Ticket

Description of Changes Made

Added Organization schema markup to the homepage using JSON-LD
Updated logo URL to use higher resolution android-chrome-512x512.png following Google's recommendations
Implemented comprehensive test coverage for Organization JSON-LD functionality
Added social media links (Bluesky, LinkedIn, Instagram) to the Organization schema
Refactored JSON-LD tests to use direct template rendering for better reliability
Cleaned up test imports and improved code organization

How to Test

Visit the homepage at /
View page source and verify the Organization JSON-LD is present in the <head> section
Check that the logo URL points to https://torchbox.com/android-chrome-512x512.png
Verify social media links are included in the sameAs array
Run the test suite: fab run-test to ensure all tests pass
Validate the schema using Google's Rich Results Test tool

Screenshots

Expand to see more

The Organization JSON-LD appears in the page source as:

<script type="application/ld+json">
{
    "@context": "https://schema.org",
    "@type": "Organization",
    "name": "Torchbox",
    "url": "https://torchbox.com/",
    "logo": "https://torchbox.com/android-chrome-512x512.png",
    "sameAs": [
        "https://bsky.app/profile/torchbox.com",
        "https://www.linkedin.com/company/torchbox",
        "https://www.instagram.com/torchboxltd/"
    ]
}
</script>

MR Checklist

Add a description of your pull request and instructions for the reviewer to verify your work.
If your pull request is for a specific ticket, link to it in the description.
Stay on point and keep it small so the merge request can be easily reviewed.
Tests and linting passes.

Unit tests

Added
Not required

Documentation

Updated build docs
Updated editor guidelines (See https://intranet.torchbox.com/torchbox-com-project-docs for the private link, for Torchbox employees only)
Updated field spec (See https://intranet.torchbox.com/torchbox-com-project-docs for the private link, for Torchbox employees only)
Not required

Browser testing

I have tested in the following browsers and environments (edit the list as required)
- Latest version of Chrome on mac
- Latest version of Firefox on mac
- Latest version of Safari on mac
- Safari on last two versions of iOS
- Chrome on last two versions of Android
Not required

Data protection

Not relevant
This adds new sources of PII and documents it and modifies Birdbath processors accordingly

Light and dark mode

I have tested the changes in both light and dark mode
The change is not relevant to dark and light mode

Accessibility

Automated WCAG 2.1 tests pass
HTML validation passes
Manual WCAG 2.1 tests completed
I have tested in a screen reader
I have tested in high-contrast mode
Any animations removed for prefers-reduced-motion
Not required

Sustainability

Images are optimised and lazy-loading used where appropriate
SVGs have been optimised
Performance and transfer of data considered
If JavaScript is needed alternatives have been considered
Not required

Pattern library

The pattern library component for this template displays correctly, and does not break parent templates
The styleguide is updated if relevant
Changes are not relevant the pattern library

JakubMastalerz · 2025-10-16T13:12:24Z

tbx/project_styleguide/templates/patterns/pages/home/home_page.html

+            "@type": "Organization",
+            "name": "Torchbox",
+            "url": "https://torchbox.com/",
+            "logo": "https://torchbox.com/apple-touch-icon.png",


Not sure what to go with here actually

switched in f5b769c

tbx/project_styleguide/templates/patterns/navigation/components/breadcrumbs-jsonld.html

Tests were failing due to URL generation issues in test environment. Direct template rendering bypasses Wagtail URL resolution problems and provides more reliable test coverage for JSON-LD functionality.

Ensures Organization schema implementation is properly validated with tests for structure, social links, logo URL, and template inclusion. Prevents regression of homepage JSON-LD functionality.

Switched from apple-touch-icon.png (180x180) to android-chrome-512x512.png (512x512) for better rich results display and Google's recommended higher resolution. Follows Google PWA standards and provides better quality for search engine rich snippets. Standards followed: - Google PWA Manifest requirements: https://developers.google.com/web/fundamentals/web-app-manifest - Schema.org Organization logo recommendations: https://schema.org/Organization - Google Rich Results guidelines: https://developers.google.com/search/docs/appearance/structured-data/logo The 512x512px resolution exceeds Google's minimum requirement of 112x112px and follows PWA best practices for high-quality icon display.

- Move all imports to top of file - Remove redundant imports from test methods - Improve code organization and readability

- Add _extract_jsonld_by_type() helper to eliminate repetitive JSON-LD extraction - Add _get_organization_jsonld() helper for cleaner test methods - Reduce test_organization_jsonld_social_links from 30+ lines to 8 lines - All tests still pass with same coverage - Much more maintainable and readable

kevinhowbrook · 2025-10-23T13:29:43Z

I've upted the PR to include tests and a more 'standard' icon.

bmispelon

I've left some suggestions for improvements, mostly in the tests.

I also notice that you're using a lot of inline imports rather than putting them all at the top of the file as is normally the case, was there a reason for that?

tbx/core/tests/test_jsonld.py

tbx/project_styleguide/templates/patterns/pages/blog/blog-posting-jsonld.html

kevinhowbrook · 2026-01-05T10:41:45Z

I've left some suggestions for improvements, mostly in the tests.

I also notice that you're using a lot of inline imports rather than putting them all at the top of the file as is normally the case, was there a reason for that?

No reason, just my rogue agent - apologies

JakubMastalerz added 2 commits October 16, 2025 13:36

add organisation schema to homepage

c0e8dec

add breadcrumb schema to base template

7bf1ab7

JakubMastalerz self-assigned this Oct 16, 2025

JakubMastalerz requested a review from bmispelon October 16, 2025 13:11

JakubMastalerz commented Oct 16, 2025

View reviewed changes

bmispelon reviewed Oct 16, 2025

View reviewed changes

tbx/project_styleguide/templates/patterns/navigation/components/breadcrumbs-jsonld.html Outdated Show resolved Hide resolved

JakubMastalerz and others added 6 commits October 16, 2025 14:55

add blogposting schema

05a019e

Fix blog JSON-LD tests to use direct template rendering

2537dc5

Tests were failing due to URL generation issues in test environment. Direct template rendering bypasses Wagtail URL resolution problems and provides more reliable test coverage for JSON-LD functionality.

Add comprehensive Organization JSON-LD test coverage

5f04de5

Ensures Organization schema implementation is properly validated with tests for structure, social links, logo URL, and template inclusion. Prevents regression of homepage JSON-LD functionality.

Clean up JSON-LD test imports

896c186

- Move all imports to top of file - Remove redundant imports from test methods - Improve code organization and readability

kevinhowbrook marked this pull request as ready for review October 23, 2025 13:29

kevinhowbrook requested a review from bmispelon October 23, 2025 13:30

kevinhowbrook changed the title ~~WIP: Add Schema.org schemas to pages~~ Adds Schema.org schemas to pages Oct 23, 2025

bmispelon reviewed Oct 29, 2025

View reviewed changes

Correct import style

b60e50e

kevinhowbrook added 10 commits January 5, 2026 10:46

use BeautifulSoup for JSON-LD parsing

cf350e6

Use built-in test client instead of render_to_string in JSON-LD tests

ea8d3cc

Use assertCountEqual to test social links list directly

dc2c3a5

Remove redundant test

2759764

Remove redundant assertions

834227a

Test breadcrumb JSON-LD against parsed dict instead of raw HTML

7d69cf4

Test against actual blog page

f772e23

Remove redundant template existence tests

d0550c7

Remove redundant test_multiple_jsonld_scripts test

f11a111

Build BlogPosting JSON-LD as Python dict instead of in template

c986e19

kevinhowbrook force-pushed the BAU/TWE-670-add-schemas branch from eca8020 to c986e19 Compare January 5, 2026 12:45

kevinhowbrook requested a review from bmispelon January 5, 2026 12:51

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Adds Schema.org schemas to pages #408

Adds Schema.org schemas to pages #408

Uh oh!

JakubMastalerz commented Oct 16, 2025 •

edited by kevinhowbrook

Loading

Uh oh!

JakubMastalerz Oct 16, 2025

Uh oh!

kevinhowbrook Oct 23, 2025

Uh oh!

Uh oh!

kevinhowbrook commented Oct 23, 2025

Uh oh!

bmispelon left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

kevinhowbrook commented Jan 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Adds Schema.org schemas to pages #408

Are you sure you want to change the base?

Adds Schema.org schemas to pages #408

Uh oh!

Conversation

JakubMastalerz commented Oct 16, 2025 • edited by kevinhowbrook Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description of Changes Made

How to Test

Screenshots

MR Checklist

Unit tests

Documentation

Browser testing

Data protection

Light and dark mode

Accessibility

Sustainability

Pattern library

Uh oh!

JakubMastalerz Oct 16, 2025

Choose a reason for hiding this comment

Uh oh!

kevinhowbrook Oct 23, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

kevinhowbrook commented Oct 23, 2025

Uh oh!

bmispelon left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

kevinhowbrook commented Jan 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

JakubMastalerz commented Oct 16, 2025 •

edited by kevinhowbrook

Loading