temporalio / temporalio/samples-python
[Bug] Update Pydantic converter example to reflect `with_child_unrestricted` changes
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 367
- Forks
- 121
- Merge moyen
- 3 j 20 h
- PR mergées (30 j)
- 11
Description
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!
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Ouvrez pydantic_converter/worker.py et lisez le TODO concernant l’utilisation de with_child_unrestricted après la correction et la publication de sdk-python issue 254. Mettez à jour l’exemple de référence pour refléter cette modification, puis vérifiez que l’exemple du convertisseur Pydantic ne dépend plus du workaround décrit dans l’issue.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- documentation
- Type d'issue
- Documentation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 55/100