search

temporalio / cli

Command-line interface for running Temporal Server and interacting with Workflows, Activities, Namespaces, and other parts of Temporal
https://docs.temporal.io/cli
MIT License
243 stars 32 forks source link

[Feature Request] `temporal schedule update` with `--calendar` option to support multiple calendars #557

Closed petrkoutnycz closed 2 months ago

petrkoutnycz commented 2 months ago

Is your feature request related to a problem? Please describe.

Currently the temporal schedule update command support setting of a single calendar only which is limiting. If I try to provide an array of them instead, it fails with JSON deserialization.

Describe the solution you'd like

I would like the update command to support setting an array of calendars. I.e. --calendar can take an array with JSON definitions.

cretz commented 2 months ago

Can you provide multiple --calendar arguments, one for each calendar you want to add?

petrkoutnycz commented 2 months ago

Will try, thanks!

cretz commented 2 months ago

This is the recommended way to provide multiple of these values. Let us know if it works well for you and we'll close this issue.

petrkoutnycz commented 2 months ago

Oh crap, it works. Shame on me. Thank you 🙏