Skip to content

docs(update): added what is infisical video to our docs#5753

Merged
jakehulberg merged 2 commits intomainfrom
docs/whatisinfisical
Mar 18, 2026
Merged

docs(update): added what is infisical video to our docs#5753
jakehulberg merged 2 commits intomainfrom
docs/whatisinfisical

Conversation

@jakehulberg
Copy link
Copy Markdown
Contributor

Context

Added "what is Infisical" YouTube video in relevant doc

Screenshots

N/a

Steps to verify the change

N/a

Type

  • Fix
  • Feature
  • Improvement
  • Breaking
  • [ X] Docs
  • Chore

Checklist

  • [x ] Title follows the conventional commit format: type(scope): short description (scope is optional, e.g., fix: prevent crash on sync or fix(api): handle null response).
  • [ x] Tested locally
  • [ x] Updated docs (if needed)
  • [x ] Updated CLAUDE.md files (if needed)
  • [ x] Read the contributing guide

@jakehulberg jakehulberg requested a review from dangtony98 March 18, 2026 20:23
@maidul98
Copy link
Copy Markdown
Collaborator

maidul98 commented Mar 18, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 18, 2026

Greptile Summary

This PR adds a responsive YouTube video embed ("What is Infisical?") to the getting-started introduction page, providing users with a visual overview of the platform before they dive into the docs.

  • Adds an introductory sentence contextualizing the video
  • Embeds the YouTube video using the standard 16:9 padding-bottom responsive wrapper pattern (paddingBottom: "56.25%")
  • allowfullscreen attribute should be allowFullScreen (camelCase) to conform to React/JSX conventions used in MDX files — the lowercase form may produce a React warning

Confidence Score: 5/5

  • This is a safe, docs-only change with no backend or security implications.
  • Single documentation file change that adds a YouTube video embed. The only issue is a minor JSX attribute casing nit (allowfullscreen vs allowFullScreen) that may produce a React warning but is unlikely to break rendering in practice. No logic, security, or API concerns.
  • No files require special attention.

Important Files Changed

Filename Overview
docs/documentation/getting-started/introduction.mdx Adds a responsive YouTube iframe embed to the introduction page. Minor JSX attribute casing issue (allowfullscreenallowFullScreen) in the iframe element.

Last reviewed commit: "Added what is infisi..."

Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Copy link
Copy Markdown
Collaborator

@dangtony98 dangtony98 left a comment

Choose a reason for hiding this comment

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

LGTM

@jakehulberg jakehulberg merged commit eb59386 into main Mar 18, 2026
7 checks passed
@jakehulberg jakehulberg deleted the docs/whatisinfisical branch March 18, 2026 20:58
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