temporalio / temporalio/samples-python
[Bug] Update Pydantic converter example to reflect `with_child_unrestricted` changes
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 367
- Fork
- 121
- Merge medio
- 3g 20h
- PR unite (30g)
- 11
Descrizione
What are you really trying to do?
I want to use Pydantic and datetime objects as input/outputs/signals/queries to workflows and activities.
Describe the bug
The recommended reference example for the Pydantic converter creates a new sandbox to work around a bug in Pydantic.
There is a TODO in the code below:
https://github.com/temporalio/samples-python/blob/main/pydantic_converter/worker.py
that says:
TODO(cretz): Use with_child_unrestricted when https://github.com/temporalio/sdk-python/issues/254 is fixed and released
The issue in question has now been merged but the example hasn't been updated.
Minimal Reproduction
https://github.com/temporalio/samples-python/blob/main/pydantic_converter/worker.py
Environment/Versions
n/a
Additional context
n/a
Thanks!
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
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.
Direzione di ricerca
Apri pydantic_converter/worker.py e leggi il TODO sull’uso di with_child_unrestricted dopo che sdk-python issue 254 è stato corretto e rilasciato. Aggiorna l’esempio di riferimento per riflettere questa modifica, quindi verifica che l’esempio del convertitore Pydantic non si basi più sul workaround descritto nell’issue.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- documentation
- Tipo di issue
- Documentazione
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 55/100