Skip to content

adds a step in TW review to check video url

Justin Farris requested to merge justin-farris-video-embed into master

Why is this change being made?

We've repeatedly missed video links that include the wrong YouTube URL format for embedding videos. This breaks the content on the main release post page and preview page.

This change adds a section to the Tech Writer Image review to double-check that the youtube URLs are correct.

Author Checklist

  • Correct MR template applied (e.g blog post)
  • Provided a concise title for the MR
  • Added a description to this MR explaining the reasons for the proposed change, per say-why-not-just-what
  • Assign this change to the correct DRI
    • If the DRI for the page/s being updated isn’t immediately clear, then assign it to your manager.
    • If your manager does not have merge rights, please ask someone to merge it AFTER it has been approved by your manager in #mr-buddies.
    • If the changes relate to any part of the project other than updates to content and/or data files please make sure to ping @gl-static-site-editor in a comment for a review and merge. For example changes to .gitlab-ci.yml, JavaScript/CSS/Ruby code or the layout files.

For help with failing pipelines reach out in #mr-buddies in Slack

Edited by Farnoosh Seifoddini

Merge request reports