Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Video embedding #5821

Merged
merged 5 commits into from
Aug 15, 2024
Merged

Video embedding #5821

merged 5 commits into from
Aug 15, 2024

Conversation

hargunkaur286
Copy link
Member

@hargunkaur286 hargunkaur286 commented Aug 15, 2024

Description

This PR fixes #

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@l5io
Copy link
Contributor

l5io commented Aug 15, 2024

🚀 Preview for commit b14627c at: https://66bdf07246a3731700f7c374--layer5.netlify.app

Copy link
Contributor

@Ashparshp Ashparshp left a comment

Choose a reason for hiding this comment

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

In mobile view, the layout needs improvement.

Signed-off-by: Hargun Kaur <[email protected]>
Signed-off-by: Hargun Kaur <[email protected]>
Signed-off-by: Hargun Kaur <[email protected]>
@l5io
Copy link
Contributor

l5io commented Aug 15, 2024

🚀 Preview for commit b6ce6bd at: https://66bdfe411499f728b103da7b--layer5.netlify.app

Copy link
Member

@vishalvivekm vishalvivekm left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks

@vishalvivekm vishalvivekm merged commit d260d92 into layer5io:master Aug 15, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants