Add test for front page meta tags

Created on 28 January 2025, 2 months ago

Follow up from 🐛 Front page meta description still references obsolete node:summary Active -- add a test for the front page meta tag config so it doesn't break again.

📌 Task
Status

Active

Component

Track: SEO

Created by

🇦🇺Australia pameeela

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @pameeela
  • 🇮🇳India Akhil Babu Chengannur

    I will try to add the tests

  • Merge request !490Resolve #3502659 "Metatag test" → (Open) created by Akhil Babu
  • 🇮🇳India Akhil Babu Chengannur

    I have added the tests but it's failing because the 'Content' meta tags are getting applied to the node set as the front page. Eg: For the front page, the og:type is configured as 'website' and for other pages it's cofigured as 'article'. After creating a node and setting it as the site's front page, the og:type is still appearing as 'article' when opening the front page. This happens only while running the tests. If I try this directly from the ui (Create a new node and set it as the front page) , front page tags are coming properly

    Also, the front page meta tag config was using 'field_image' to get the image meta tags like image_src, og:image, etc. But the default homepage is a basic page, and the basic page content type has 'field_featured_image' instead of 'field_image'. I’m not sure if it was a mistake or was intentionally set (since 'field_image' field exists in the article content type), but I’ve updated it in this PR as well.

  • 🇮🇳India Akhil Babu Chengannur

    Moving to 'Needs review' so the maintainers can provide their insights on #4.

  • Pipeline finished with Failed
    2 months ago
    Total: 2367s
    #415565
  • 🇮🇳India Akhil Babu Chengannur

    Hmm.. in my local only the 'og:type' validation part of the test if failing. But here it's failing while opening the front page 🧐

  • 🇺🇸United States phenaproxima Massachusetts

    🤔 I'd like to take a crack at this; I have a slightly different approach in mind that should (hopefully) not require a ton of refactoring in the test.

  • 🇺🇸United States thejimbirch Cape Cod, Massachusetts

    Moving to Needs Work for the comment in #7

Production build 0.71.5 2024