Closed cretz closed 2 months ago
I have updated the proposal with the following:
Temporalio::SearchAttributes
as both the collection and class under which all nested search attribute things resideTemporalio::Metric
as both the base metric and class under which all nested metric things resideOptions
structs on client/connectionCloudOperationsClient
namespace
a required field instead of defaulting to default
Temporalio::Converter
module to Temporalio::Converters
Temporalio::Converters::PayloadConverter
to be a simple base class instead of having it impl composite, and created Temporalio::Converters::PayloadConverter::Composite
Closing this PR in favor of https://github.com/temporalio/proposals/pull/93 for a fresh round of comments. Will link back to this one for posterity, but unaware of any unresolved issues here. Anyone here can feel free to comment on https://github.com/temporalio/proposals/pull/93.
⚠️ Next PR round opened at #93
Summary