Skip to content

[Fix] Snapshot Schedules Update: make paused field optional#266

Merged
alessandroargentieri merged 1 commit into
masterfrom
fix/snapshot-schedules-paused-optional
Jun 12, 2025
Merged

[Fix] Snapshot Schedules Update: make paused field optional#266
alessandroargentieri merged 1 commit into
masterfrom
fix/snapshot-schedules-paused-optional

Conversation

@giornetta

Copy link
Copy Markdown
Member

This MR makes the Paused field in UpdateSnapshotScheduleRequest nullable, since it's an optional parameter and its zero value false is not invalid.

@giornetta giornetta self-assigned this Jun 12, 2025
@giornetta giornetta added the bug Something isn't working label Jun 12, 2025
@giornetta giornetta changed the title [Fix]Snapshot Schedules Update: make paused field optional [Fix] Snapshot Schedules Update: make paused field optional Jun 12, 2025

@alessandroargentieri alessandroargentieri left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alessandroargentieri alessandroargentieri merged commit 639be60 into master Jun 12, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants