Skip to content

Update meta in migration pages#1774

Merged
Alek99 merged 1 commit intomainfrom
carlos/update-migration-metas
Mar 11, 2026
Merged

Update meta in migration pages#1774
Alek99 merged 1 commit intomainfrom
carlos/update-migration-metas

Conversation

@carlosabadia
Copy link
Collaborator

No description provided.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 11, 2026

Greptile Summary

This PR improves SEO for the four migration landing pages by updating meta titles and descriptions to be more keyword-rich and specific, and by injecting FAQPage JSON-LD structured data into each page. A new faq_jsonld helper is added to meta.py to serialize a pre-built schema dict into a <script type="application/ld+json"> tag, consistent with the existing blog_jsonld, pricing_jsonld, and website_organization_jsonld helpers.

Key changes:

  • pcweb/meta/meta.py: New faq_jsonld(faq_schema: dict) helper added after blog_index_jsonld.
  • Four migration pages (low_code, no_code, other_ai_tools, other_frameworks): each gains a module-level FAQ schema constant and calls faq_jsonld() as the first child of the page's root div.
  • other_ai_tools and other_frameworks pages also swap their OG preview image from low_code_migration_preview.webp to index_preview.webp, correcting what appeared to be a copy-paste error from the original.
  • The faq_jsonld helper differs slightly from peer helpers — it accepts an already-assembled dict rather than individual typed parameters — but this is a reasonable design choice given the verbosity of FAQ schemas.

Confidence Score: 5/5

  • This PR is safe to merge — all changes are additive SEO/metadata updates with no runtime risk.
  • All FAQ schemas are module-level Python constants containing only serializable strings and dicts, so json.dumps cannot fail at runtime. The new faq_jsonld helper is a one-liner with no branching logic. Meta title/description updates are pure strings. No business logic, state, database, or API calls are touched.
  • No files require special attention.

Important Files Changed

Filename Overview
pcweb/meta/meta.py Adds a simple faq_jsonld helper that serializes a pre-built schema dict into a JSON-LD script tag; straightforward and consistent with existing JSON-LD helpers.
pcweb/pages/migration/low_code/init.py Updates meta title/description for SEO and injects a FAQPage JSON-LD block via faq_jsonld; the schema is a well-formed module-level constant.
pcweb/pages/migration/no_code/init.py Same pattern as low_code: updated meta and injected FAQPage JSON-LD for no-code migration page.
pcweb/pages/migration/other_ai_tools/init.py Same pattern: updated meta (including swapping preview image from low_code_migration_preview.webp to index_preview.webp) and added FAQPage JSON-LD.
pcweb/pages/migration/other_frameworks/init.py Same pattern: updated meta (also swaps to index_preview.webp) and added FAQPage JSON-LD for other-frameworks migration page.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["Module-level FAQ schema constant\n(e.g. LOW_CODE_FAQ_SCHEMA dict)"] --> B["faq_jsonld(faq_schema)"]
    B --> C["json.dumps(faq_schema)"]
    C --> D["rx.el.script(..., type='application/ld+json')"]
    D --> E["Rendered inside page's top-level rx.el.div"]
    E --> F["Browser / Search Engine reads\nFAQPage structured data"]

    subgraph meta.py
        B
        C
        D
    end

    subgraph migration page __init__.py
        A
        E
    end
Loading

Last reviewed commit: 48d5311

@Alek99 Alek99 self-requested a review March 11, 2026 20:40
@Alek99 Alek99 merged commit 933085e into main Mar 11, 2026
10 checks passed
@Alek99 Alek99 deleted the carlos/update-migration-metas branch March 11, 2026 20:41
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