Return updated column on PUT /boards/:id/columns/:id

[robzolkos]

Summary

The JSON response for PUT /:account_slug/boards/:board_id/columns/:column_id was 204 No Content, forcing clients to follow up with a GET to see their own update. The Smithy contract that the SDKs are generated from already declares UpdateColumn returns a Column, so the server was out of sync with the documented shape.

Rendering the show template on the JSON path makes the update return the same payload as GET /:account_slug/boards/:board_id/columns/:column_id. The Turbo Stream path is unchanged.

Discovered during a fizzy-cli QA sweep. Same class of issue as #2848 (boards) and #2849 (card move).

Changes

• app/controllers/boards/columns_controller.rb — format.json { head :no_content } → format.json { render :show }
• test/controllers/boards/columns_controller_test.rb — "update as JSON" test now asserts the returned body
• docs/api/sections/columns.md — replaced "Returns 204 No Content" with the 200 response shape

Mobile App check

• Neither mobile app has native client code calling PUT /boards/:id/columns/:id
• Neither mobile app has code paths that depend on this endpoint returning 204 No Content
• Neither mobile app is affected by this response change to 200 OK with the updated column

[claude]

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

[Copilot]

Pull request overview

Tip

If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.

This PR brings the server behavior for updating board columns back in sync with the documented/contracted API shape by returning the updated Column JSON payload on successful PUT /:account_slug/boards/:board_id/columns/:column_id, instead of 204 No Content.

Changes:

• Update JSON response for column updates to render :show (returns the updated column payload).
• Expand controller test coverage to assert the JSON response body for the update endpoint.
• Update API documentation to describe the 200 OK response and payload shape.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
app/controllers/boards/columns_controller.rb	Changes JSON update response from 204 to rendering the existing show JSON shape.
test/controllers/boards/columns_controller_test.rb	Updates “update as JSON” test to assert 200 + returned id/name.
docs/api/sections/columns.md	Documents 200 OK with updated column payload for the PUT endpoint.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.