Invalid reference to Temporal::Api::Enums::V1::TaskQueueType::Workflow
Nobody has claimed this yet.
- #194 by @cdimitroulas — closed without merging
Assessment
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Newbie friendliness
- 55/100
Research direction
Start at Temporal::Connection::GRPC#describe_task_queue and inspect the enum reference shown in the issue. Run the supplied connection.describe_task_queue example after the change and confirm that it completes without the reported NameError.
Written by the indexing model from the issue text.
Description
In the Temporal::Connection::GRPC#describe_task_queue method there is a reference to this enum -> Temporal::Api::Enums::V1::TaskQueueType::Workflow
A simple bit of code like this seems to suggest that this is an invalid reference as I get an error:
connection = Temporal::Connection.generate(Temporal.configuration.for_connection)
puts connection.describe_task_queue(namespace: Temporal.configuration.namespace, task_queue: Temporal.configuration.task_queue)
The error:
NameError: uninitialized constant Temporal::Api::Enums::V1::TaskQueueType::Workflow
Did you mean? Temporal::Workflow
Temporal::Api::Workflow
I believe that the correct reference should be Temporal::Api::Enums::V1::TaskQueueType::TASK_QUEUE_TYPE_WORKFLOW. When using that instead, then the code above runs as expected. Happy to open a PR if this is indeed the correct fix.
- Dominant language
- Ruby
- Stars
- 287
- Forks
- 113
- PR merge metrics
- No merged PRs in 30d
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from coinbase/temporal-ruby
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
coinbase/temporal-ruby#361 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
coinbase/temporal-ruby#341 ·
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
coinbase/temporal-ruby#328 · 1 comment ·
-
NoMethodError when await_workflow_result is called for long running workflowsPossibly taken @iekundayo claimed this 717 days ago. Open
Difficulty 3/5 1-2 days Newbie friendliness 45/100
coinbase/temporal-ruby#326 · 1 comment ·
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
coinbase/temporal-ruby#324 · 3 comments · 2 reactions ·
All issues in coinbase/temporal-ruby
Similar issues
-
good first issue
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 73/100
DataDog/datadog-lambda-rb#167 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
alphagov/public-asset-checker#378 · 1 comment ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
betagouv/acces-cible#786 ·
Maintainers usually reply within 1 day
-
data error
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
riscv/riscv-unified-db#2749 ·
Maintainers usually reply within 1 day