ReviewOS

also looking at this

meemalabs/laravel-media-converter

fix: restore media_id

#20
Merged glennmichael123 wants to merge restore-mediaid into main
1 file +16 -6

Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.

src/Jobs/CreateVideoConversion.phpmodified+1-1
Changes to src/Jobs/CreateVideoConversion.php
@@ -35,7 +35,7 @@ class CreateVideoConversion implements ShouldQueue
3535 * @param array $tags
3636 * @param int $mediaId
3737 */
38 public function __construct($jobSettings, $tags, $mediaId = null)
38 public function __construct($jobSettings, $tags = [], $mediaId = null)
3939 {
4040 $this->jobSettings = $jobSettings;
4141 $this->tags = $tags;