Skip to content

removed vtt file and video script - #84

Merged
Pasha Zayko (pasha-zayko) merged 1 commit into
mainfrom
jtd/mi-and-installation-updates-3
Aug 28, 2026
Merged

removed vtt file and video script#84
Pasha Zayko (pasha-zayko) merged 1 commit into
mainfrom
jtd/mi-and-installation-updates-3

Conversation

@jtdauria-shi

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI lite review requested due to automatic review settings August 28, 2026 20:27
@jtdauria-shi jtdauria-shi self-assigned this Aug 28, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the SHIELD installation video embed (including the custom chapter list and JavaScript seek/play handler) from the installation prerequisites documentation and deletes the associated WebVTT chapters file.

Changes:

  • Removed the “Installation Video” section from Installation.md (video tag, chapter buttons list, and inline script).
  • Deleted shield_installation-chapters.vtt from docs/assets/Videos/.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
docs/SHIELD/Prerequisites/Installation.md Removes the embedded installation video UI and related inline script from the prerequisites page.
docs/assets/Videos/shield_installation-chapters.vtt Deletes the chapter track file that was previously used by the removed video embed.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@pasha-zayko
Pasha Zayko (pasha-zayko) merged commit cc41f96 into main Aug 28, 2026
5 checks passed
@pasha-zayko
Pasha Zayko (pasha-zayko) deleted the jtd/mi-and-installation-updates-3 branch August 28, 2026 20:30
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.

3 participants