Typed input and output
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 25/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- ruby
- Lĩnh vực
- backend, distributed-systems
Hướng nghiên cứu
Bắt đầu với lib/temporal/concerns/typed.rb tại dòng 22 và xem xét cách các activity và workflow hiện đang xử lý việc tuần tự hóa đầu vào và đầu ra. Issue này cần một cách tiếp cận đã được thống nhất cho việc xác thực dry-struct có thể tái sử dụng, đầu ra có kiểu và việc xác thực trước khi kích hoạt workflow; phần hoàn thành cần bao gồm một API được định nghĩa và các test cho từng hành vi.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Hi! I am trying to get rid of JSON.parse(input) and JSON.dump(output) in activities/workflows and wanted to define Input and Output classes with dry-struct to define the structure.
Then I found this line in https://github.com/coinbase/temporal-ruby/blob/master/lib/temporal/concerns/typed.rb#L22
unless klass.is_a?(Dry::Types::Type)
Now I am stuck on how to make reusable structs with validation for workflow/activity params. I have three questions:
- Does it make sense to change condition in typed.rb to allow dry structs?
- How I can add typed output, so I don't need to serialize and deserialize outputs multiple time?
- Any best practices to validate inputs before activating the workflow?
Any ideas how to solve this?
- Ngôn ngữ chính
- Ruby
- Star
- 288
- Fork
- 113
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của coinbase/temporal-ruby
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
coinbase/temporal-ruby#361 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
coinbase/temporal-ruby#341 ·
-
Emitting Metrics for PrometheusĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
coinbase/temporal-ruby#328 · 1 bình luận ·
-
NoMethodError when await_workflow_result is called for long running workflowsCó thể đã có người làm @iekundayo đã nhận 710 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
coinbase/temporal-ruby#326 · 1 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
coinbase/temporal-ruby#324 · 3 bình luận · 2 reaction ·
Tất cả issue của coinbase/temporal-ruby
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 8 ngày
-
bug: "Conversation information" accordion renders blank with 0px height when conversation has no attributesCó thể đã có người làm @gihovani đã nhận hôm nay. Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 2 ngày
-
`hanami generate repository` would be a nice alias for `hanami generate repo`Có thể đã có người làm @sandbergja đã nhận hôm nay. Đang mởbug help wanted
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
hanami/hanami-cli#455 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 69/100
asciidoctor/asciidoctor#4882 · 4 bình luận ·