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+3-3
Changes to src/Jobs/CreateVideoConversion.php
@@ -31,9 +31,9 @@ class CreateVideoConversion implements ShouldQueue
3131 /**
3232 * Create a new job instance.
3333 *
34 * @param $jobSettings
35 * @param $tags
36 * @param $mediaId
34 * @param array $jobSettings
35 * @param array $tags
36 * @param int $mediaId
3737 */
3838 public function __construct($jobSettings, $tags, $mediaId = null)
3939 {