Skip to content

Support forSaleMode when creating or updating a workspace - #282

Merged
mroloux merged 1 commit into
masterfrom
matti/seats-1795-support-forsalemode-when-creating-or-updating-a-workspace-in
Oct 22, 2026
Merged

mroloux merged 1 commit into
masterfrom
matti/seats-1795-support-forsalemode-when-creating-or-updating-a-workspace-in

Conversation

@mroloux

@mroloux mroloux commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

https://linear.app/seatsio/issue/SEATS-1795/support-forsalemode-when-creating-or-updating-a-workspace-in-the-sdks

Issue: The API now accepts an optional forSaleMode (RATE_LIMITED or UNLIMITED_BILLABLE) when creating or updating a workspace (SEATS-1785), but the SDK can't send it.

Fix: Add an optional forSaleMode parameter to create and update workspace. Leaving it out keeps the current behaviour.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 22, 2026 12:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The changes preserve compatibility and correctly include the optional mode, with no blocking defects identified; integration tests were not executed.

No open issues 🎉

What changed in this PR

Adds optional forSaleMode support to workspace creation and updates in the Java SDK while preserving existing method signatures and behavior when omitted.

Changes:

  • Adds ForSaleMode with RATE_LIMITED and UNLIMITED_BILLABLE values.
  • Adds create and update overloads that include the mode only when provided.
  • Adds integration tests exercising both operations with UNLIMITED_BILLABLE.
File Description
src/​test/​java/​seatsio/​workspaces/​UpdateWorkspaceTest.java Exercises updating a workspace with a sale mode.
src/​test/​java/​seatsio/​workspaces/​CreateWorkspaceTest.java Exercises creating a workspace with a sale mode.
src/​main/​java/​seatsio/​workspaces/​Workspaces.java Adds backward-compatible overloads and optional request serialization.
src/​main/​java/​seatsio/​workspaces/​ForSaleMode.java Defines the supported sale modes.

Review effort: Balanced

@mroloux
mroloux marked this pull request as ready for review October 22, 2026 13:07
@mroloux
mroloux requested a review from a team as a code owner October 22, 2026 13:07
@mroloux
mroloux requested review from bverbeken and schaloner and removed request for a team October 22, 2026 13:07
@mroloux
mroloux merged commit 291f08a into master Oct 22, 2026
3 checks passed
@mroloux
mroloux deleted the matti/seats-1795-support-forsalemode-when-creating-or-updating-a-workspace-in branch October 22, 2026 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants