Closed cretz closed 1 month ago
Per https://github.com/temporalio/api/pull/308, https://github.com/temporalio/sdk-dotnet/blob/23af80339e84c1defc35fe61fbe053b26709011d/src/Temporalio/Client/Schedules/SchedulePolicy.cs#L22 should now be 1 year. Also, consider if we can have defaults be server-side defaults instead of client-side (changes types to make it seem like this value is optional when it's not). Will discuss with team.
For now it was decided to just change this default value to match server default
Describe the solution you'd like
Per https://github.com/temporalio/api/pull/308, https://github.com/temporalio/sdk-dotnet/blob/23af80339e84c1defc35fe61fbe053b26709011d/src/Temporalio/Client/Schedules/SchedulePolicy.cs#L22 should now be 1 year. Also, consider if we can have defaults be server-side defaults instead of client-side (changes types to make it seem like this value is optional when it's not). Will discuss with team.