ReviewOS

also looking at this

meemalabs/laravel-media-converter

feat: add support for tagging

#17
Merged glennmichael123 wants to merge add-tags into main
4 files +20 -5

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+5-1
Changes to src/Jobs/CreateVideoConversion.php
@@ -23,13 +23,17 @@ class CreateVideoConversion implements ShouldQueue
2323 * @param $jobSettings
2424 * @param $mediaId
2525 */
26 public function __construct($jobSettings, $mediaId = null)
26 public function __construct($jobSettings, $mediaId = null, $teamId = null)
2727 {
2828 $this->jobSettings = $jobSettings;
2929
3030 if ($mediaId) {
3131 $this->mediaId = $mediaId;
3232 }
33
34 if ($teamId) {
35 $this->teamId = $teamId;
36 }
3337 }
3438
3539 /**