temporalio / temporalio/samples-python
[Bug] Update Pydantic converter example to reflect `with_child_unrestricted` changes
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 367
- Forks
- 121
- Ø Merge
- 3 T. 20 Std.
- Gemergte PRs (30 T.)
- 11
Beschreibung
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!
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Öffne pydantic_converter/worker.py und lies den TODO zum Verwenden von with_child_unrestricted, nachdem sdk-python issue 254 behoben und veröffentlicht wurde. Aktualisiere das Referenzbeispiel, um diese Änderung widerzuspiegeln, und überprüfe anschließend, dass das Pydantic-Converter-Beispiel nicht mehr auf den im Issue beschriebenen Workaround angewiesen ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- documentation
- Issue-Typ
- Dokumentation
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- Unter einer Stunde
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 55/100