Title: [Bug] Update Pydantic converter example to reflect `with_child_unrestricted` changes · Issue #126 · temporalio/samples-python · GitHub
Open Graph Title: [Bug] Update Pydantic converter example to reflect `with_child_unrestricted` changes · Issue #126 · temporalio/samples-python
X Title: [Bug] Update Pydantic converter example to reflect `with_child_unrestricted` changes · Issue #126 · temporalio/samples-python
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 sandb...
Open Graph 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 th...
X 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 th...
Opengraph URL: https://github.com/temporalio/samples-python/issues/126
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"[Bug] Update Pydantic converter example to reflect `with_child_unrestricted` changes","articleBody":"### What are you really trying to do?\r\n\r\nI want to use Pydantic and datetime objects as input/outputs/signals/queries to workflows and activities.\r\n\r\n\u003c!-- \r\nTell us at a high level what you're doing, to avoid XY problem (https://en.wikipedia.org/wiki/XY_problem) \r\n--\u003e\r\n\r\n### Describe the bug\r\n\r\nThe recommended reference example for the Pydantic converter creates a new sandbox to work around a bug in Pydantic.\r\n\r\nThere is a TODO in the code below:\r\n\r\nhttps://github.com/temporalio/samples-python/blob/main/pydantic_converter/worker.py\r\n\r\nthat says:\r\n\r\n\u003e TODO(cretz): Use with_child_unrestricted when https://github.com/temporalio/sdk-python/issues/254 is fixed and released\r\n\r\nThe [issue in question](https://github.com/temporalio/sdk-python/issues/254) has now been merged but the example hasn't been updated.\r\n\r\n\r\n\r\n### Minimal Reproduction\r\n\r\nhttps://github.com/temporalio/samples-python/blob/main/pydantic_converter/worker.py\r\n\r\n\u003c!-- \r\nModify our hello world templates to demonstrate:\r\n\r\n- TypeScript: https://github.com/temporalio/samples-typescript/tree/main/hello-world\r\n- Go: https://github.com/temporalio/money-transfer-project-template-go\r\n- Java: https://github.com/temporalio/money-transfer-project-template-java\r\n- PHP: https://github.com/temporalio/samples-php#samples\r\n--\u003e\r\n\r\n### Environment/Versions\r\n\r\nn/a\r\n\r\n### Additional context\r\n\r\nn/a\r\n\r\n\r\nThanks!","author":{"url":"https://github.com/gregbrowndev","@type":"Person","name":"gregbrowndev"},"datePublished":"2024-06-18T17:22:24.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":6},"url":"https://github.com/126/samples-python/issues/126"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:868d4aad-d737-c789-871b-89298e0e4d15 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | BC6A:5CED0:3A87DA:4ED224:6A4D2EF0 |
| html-safe-nonce | 4792c77e760eb84ccd4d6ad707069cea85185c64726d238df28fced2e7d214d0 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCQzZBOjVDRUQwOjNBODdEQTo0RUQyMjQ6NkE0RDJFRjAiLCJ2aXNpdG9yX2lkIjoiNjY4MTk2MTgzMTczMDUwMDk2IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | 21df3ac288364a0755e4d945e6746e266d1327fba656d0437dec065f97f72d01 |
| hovercard-subject-tag | issue:2398236949 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/temporalio/samples-python/126/issue_layout |
| twitter:image | https://opengraph.githubassets.com/66082954cea8468dca013dab3b0f94c738b0fdec9808f2dd83f3371971939a17/temporalio/samples-python/issues/126 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/66082954cea8468dca013dab3b0f94c738b0fdec9808f2dd83f3371971939a17/temporalio/samples-python/issues/126 |
| og:image:alt | 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 th... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | gregbrowndev |
| hostname | github.com |
| expected-hostname | github.com |
| None | 92571a8944142227b7e19cd10918b1ddd06e5066c1ad5bc7e4769cf6140a87e6 |
| turbo-cache-control | no-preview |
| go-import | github.com/temporalio/samples-python git https://github.com/temporalio/samples-python.git |
| octolytics-dimension-user_id | 56493103 |
| octolytics-dimension-user_login | temporalio |
| octolytics-dimension-repository_id | 454557884 |
| octolytics-dimension-repository_nwo | temporalio/samples-python |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 454557884 |
| octolytics-dimension-repository_network_root_nwo | temporalio/samples-python |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 93f17a978ee60bc4668e1d7b90e6bd2d622261fd |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width