-
Notifications
You must be signed in to change notification settings - Fork 1
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
adjust job-id params and namespace for sending files to s3 #1373
base: main
Are you sure you want to change the base?
Conversation
@cocina_object = cocina_object | ||
@workflow_context = workflow_context | ||
@bare_druid = cocina_object.externalIdentifier.delete_prefix('druid:') | ||
@logger = logger || Logger.new($stdout) | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this class never uses the logger, so remove for now unless we specifically need it later (it came as copypasta from the similar Ocr class)
end | ||
end | ||
|
||
private | ||
|
||
def sttable_filenames | ||
Dor::TextExtraction::SpeechToText.new(cocina_object:, workflow_context: workflow.context).filenames_to_stt | ||
Dor::TextExtraction::SpeechToText.new(cocina_object:).filenames_to_stt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the filenames_to_stt
method doesn't need workflow_context so don't bother passing it in
9ebbfc1
to
1fa3e60
Compare
@@ -27,8 +27,7 @@ def perform_work | |||
def send_sqs_message | |||
message_body = { | |||
id: job_id, | |||
druid:, | |||
media: | |||
druid: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
leaving the druid in the job message (even though it happens to be included in the job_id now) ... maybe useful for logging or other purposes to have it distinct
1fa3e60
to
7cb472c
Compare
Why was this change made? 🤔
Fixes #1371 - job id is now druid-version and so is the place where we put the files in S3 bucket
How was this change tested? 🤨
Spec