Skip to content

Update SEP SEO #6037

Olivia Madrid requested to merge feat/img-caption-seo-f6037 into master

Ticket(s) / Related Merge Requests

#6037 (closed)

Summary of Changes

Made updates to the meta tags for image/multi-image posts as described in #6037 (closed).

  • add alt tags to each image with the auto_caption for that image
  • Meta
    • author/og:author -- is now the poster's username, not 'Minds'
    • description/og:description
      • don't append descriptions with "subscribe to @channel" anymore
      • If there's no description text already included with the post, start the description with "Image from @channel."
      • If there's an auto_caption, append the description with a string of text with the AI caption (formatted as per example in the table on #6037 (closed))

There are a lot of little rules going on with how we are currently handling meta description fields based on available data, and the table in #6037 (closed) didn't account for all of the potential variations. I used my best judgment to accomplish the goal.

Testing Considerations

SEP meta tags and image alt attributes for image/multi-image posts

Deployment Considerations

There is a bug with the auto_caption field being returned for multi-image posts. This probably shouldn't be merged until it's fixed. See comments below for discussion.

Regression Scope

SEP SEO, SEO og:title metadata for all pages

Platform Affected (web, mobile, etc)

Web

Developer Testing Completed

  • Meta tags for image posts and multi-image posts have been updated as described in the table on #6037 (closed), with one exception - I didn't add '| Minds' to the end of the `og:title` field, the same way it works for the `title` field. There are a number of things that are factored into the og:title right now so it wasn't super straightforward. @markryansallee if this is vital, let me know and I'll walk you through what's going on and we can find a solution. UPDATE after discussion: " | Minds" is now added to the end of all og:titles.
  • Alt tags with AI captions added to images (and each image in multi-image posts)

👀 NOTE 👀 Due to a semicolon-related bug with the auto_caption field for multi-image posts, I used this fake data for testing: "a drawing of a circus tent with people in it; the circus is a sex circus; illustration; a painting with orange and yellow colors; the painting is a sexy image; abstract" ... That is what is used in the screenshots below. If we test before this is fixed, the caption strings will be incorrect.

Screenshots / Screen Recording

image image image

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out
Edited by Olivia Madrid

Merge request reports