Typed input and output
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- ruby
- Ambito
- backend, distributed-systems
Direzione di ricerca
Inizia da lib/temporal/concerns/typed.rb alla riga 22 e verifica come activities e workflows gestiscono attualmente la serializzazione di input e output. L’issue richiede un approccio condiviso per la validazione riutilizzabile con dry-struct, gli output tipizzati e la validazione prima dell’attivazione del workflow; il lavoro sarà completo quando includerà un’API definita e test per ogni comportamento.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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?
- Lingua principale
- Ruby
- Stelle
- 288
- Fork
- 113
- Merge medio
- 10g 15h
- PR unite (30g)
- 2
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di coinbase/temporal-ruby
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
coinbase/temporal-ruby#361 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
coinbase/temporal-ruby#341 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
coinbase/temporal-ruby#328 · 1 commento ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
coinbase/temporal-ruby#326 · 1 commento ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
coinbase/temporal-ruby#324 · 3 commenti · 2 reazioni ·
Tutte le issue di coinbase/temporal-ruby
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
riscv/riscv-unified-db#2626 ·
-
Component: GLib
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
ds-drift
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
we-promise/sure#3693 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
simp/pupmod-simp-simp#395 ·