Skip to content

fix(API): branching: align operations with the backend behavior#1136

Merged
Sven Dunemann (forelabs) merged 2 commits into
mainfrom
docs/branches-align-controller-behavior
May 20, 2026
Merged

fix(API): branching: align operations with the backend behavior#1136
Sven Dunemann (forelabs) merged 2 commits into
mainfrom
docs/branches-align-controller-behavior

Conversation

@forelabs
Copy link
Copy Markdown
Member

Aligns the Branches operations and branch schema with the actual behavior of Api::V2::BranchesController:

  • Add 401, 403, 422, and 409 responses where reachable across all Branches endpoints; 403 descriptions note the specific conditions (OAuth scope, Pundit, restrict_feature!(:branching))
  • Document asynchronous semantics for create, merge, and sync: background jobs, transitional states, empty response bodies for merge/sync, and guidance to poll until state settles
  • Add base field to create request body and branch schema (nullable, newer-branching system only)
  • Declare strategy as an enum (use_main, use_branch) in merge and sync
  • Enumerate the seven public-facing state values in branch schema
  • Note that the Branching account feature must be enabled on every operation; clarify that update only accepts name

Aligns the Branches operations and branch schema with the actual
behavior of Api::V2::BranchesController:

- Add 401, 403, 422, and 409 responses where reachable across all
  Branches endpoints; 403 descriptions note the specific conditions
  (OAuth scope, Pundit, restrict_feature!(:branching))
- Document asynchronous semantics for create, merge, and sync:
  background jobs, transitional states, empty response bodies for
  merge/sync, and guidance to poll until state settles
- Add base field to create request body and branch schema (nullable,
  newer-branching system only)
- Declare strategy as an enum (use_main, use_branch) in merge and sync
- Enumerate the seven public-facing state values in branch schema
- Note that the Branching account feature must be enabled on every
  operation; clarify that update only accepts name
@jablan jablan changed the title docs(branches): align operations with controller behavior fix(API): branching: align operations with the backend behavior May 20, 2026
Copy link
Copy Markdown
Collaborator

@jablan jablan left a comment

Choose a reason for hiding this comment

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

I altered the title a bit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants