Skip to content

Make channels.update use the update options pattern#226

Merged
tudddorrr merged 1 commit into
developfrom
channel-update-options
Jul 21, 2026
Merged

Make channels.update use the update options pattern#226
tudddorrr merged 1 commit into
developfrom
channel-update-options

Conversation

@tudddorrr

@tudddorrr tudddorrr commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Channel update API consolidation

  • Collapsed two separate request classes (ChannelsUpdateRequest, ChannelsUpdateOwnerRequest) and a branching update method into a single UpdateChannelOptions parameter with nullable fields, simplifying the call site and eliminating internal conditional logic

Partial update support

  • Replaced UnityEngine.JsonUtility with a new JsonUtils.BuildObject helper that omits null fields from serialized JSON, enabling selective partial updates (only changed fields are sent) rather than always sending every field

API ergonomics improvement

  • Changed Channel.Update() from accepting positional parameters to a single options object, making callers more readable and less brittle when adding/removing update fields

@tudddorrr tudddorrr added the breaking This pull request contains a breaking change label Jul 21, 2026
@tudddorrr
tudddorrr merged commit eab934c into develop Jul 21, 2026
3 checks passed
@tudddorrr
tudddorrr deleted the channel-update-options branch July 21, 2026 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking This pull request contains a breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant