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

add ffmpeg to support videos, when using Spatie media-library for Videos #671

Merged
merged 1 commit into from
Feb 20, 2024

Conversation

negoziator
Copy link
Contributor

This PR makes it possible to use Spatie Media lbrary for videos.

In extension to this pr - which allows SVG. This PR just allows Videos as well 😄

Motivation

So if anyone uses Spatie Media library - they have an option to install the php-ffmpeg/php-ffmpeg package, which enables support for e.g. rendering a video thumbnail alongside with seamless making a <video>-tag for playback.

See docs: here

This package requires ffmpeg to be installed on the system, so instead of forcing people to publish sail, i consider the extension to be valuable enough for being preinstalled in Sail.

@taylorotwell taylorotwell merged commit a05861c into laravel:1.x Feb 20, 2024
5 checks passed
itinerare pushed a commit to itinerare/Alcyone that referenced this pull request Feb 26, 2024
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [laravel/sail](https://github.com/laravel/sail) | `1.27.4` -> `1.28.0` | [![age](https://developer.mend.io/api/mc/badges/age/packagist/laravel%2fsail/1.28.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/packagist/laravel%2fsail/1.28.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/packagist/laravel%2fsail/1.27.4/1.28.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/packagist/laravel%2fsail/1.27.4/1.28.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>laravel/sail (laravel/sail)</summary>

### [`v1.28.0`](https://github.com/laravel/sail/blob/HEAD/CHANGELOG.md#v1280---2024-02-20)

[Compare Source](laravel/sail@v1.27.4...v1.28.0)

-   Changing pcov Directory by [@&#8203;joaopalopes24](https://github.com/joaopalopes24) in laravel/sail#670
-   add ffmpeg to support videos, when using Spatie media-library for Videos by [@&#8203;negoziator](https://github.com/negoziator) in laravel/sail#671

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yMTQuMCIsInVwZGF0ZWRJblZlciI6IjM3LjIxNC4wIiwidGFyZ2V0QnJhbmNoIjoiZGV2ZWxvcCJ9-->

Reviewed-on: https://code.itinerare.net/itinerare/Alcyone/pulls/7
Co-authored-by: Amadeus[bot] <amadeus@itinerare.net>
Co-committed-by: Amadeus[bot] <amadeus@itinerare.net>
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.

None yet

2 participants