Skip to content

Commit

Permalink
WEB-6582: Missed a reference to the new width_required argument
Browse files Browse the repository at this point in the history
Also set a default of true, so we'd get a different error in future.
  • Loading branch information
sammyd committed Oct 16, 2023
1 parent 487d16e commit 847a9b4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion app/lib/image_provider/provider.rb
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ class Provider

attr_reader :extractor, :width_required

def initialize(extractor:, width_required:)
def initialize(extractor:, width_required: true)
@extractor = extractor
@width_required = width_required
end
Expand Down
2 changes: 1 addition & 1 deletion app/lib/runner/base.rb
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ def upload_video_course(release_file:)
video_course = parser.parse

image_extractor = ImageProvider::VideoCourseExtractor.new(video_course)
image_provider = ImageProvider::Provider.new(extractor: image_extractor)
image_provider = ImageProvider::Provider.new(extractor: image_extractor, width_required: false)
image_provider.process
Renderer::VideoCourse.new(video_course, image_provider:).render
Api::Betamax::VideoCourseUploader.upload(video_course)
Expand Down

0 comments on commit 847a9b4

Please sign in to comment.