X Tutup
Skip to content

fix: register task pause/resume routes under /api/v2#22544

Merged
EhabY merged 1 commit intomainfrom
move-pause-resume-stable-api
Mar 3, 2026
Merged

fix: register task pause/resume routes under /api/v2#22544
EhabY merged 1 commit intomainfrom
move-pause-resume-stable-api

Conversation

@EhabY
Copy link
Contributor

@EhabY EhabY commented Mar 3, 2026

The pause/resume endpoints were only registered under /api/experimental but the frontend and Go SDK were calling /api/v2, resulting in 404s. Register the routes in the v2 group, update the SDK client paths, and fix swagger annotations (Accept → Produce) since these POST endpoints have no request body.

@EhabY EhabY self-assigned this Mar 3, 2026
@EhabY EhabY force-pushed the move-pause-resume-stable-api branch from 3ea8019 to 38d47b2 Compare March 3, 2026 12:20
The pause/resume endpoints were only registered under
/api/experimental but the frontend and Go SDK were calling
/api/v2, resulting in 404s. Register the routes in the v2
group, update the SDK client paths, and fix swagger annotations
(Accept → Produce) since these POST endpoints have no request body.
@EhabY EhabY force-pushed the move-pause-resume-stable-api branch from 38d47b2 to 6088d9a Compare March 3, 2026 12:40
@EhabY EhabY merged commit 9d2aed8 into main Mar 3, 2026
47 of 49 checks passed
@EhabY EhabY deleted the move-pause-resume-stable-api branch March 3, 2026 13:34
@github-actions github-actions bot locked and limited conversation to collaborators Mar 3, 2026
@matifali
Copy link
Member

matifali commented Mar 3, 2026

What happens to old /experiential routes? Do they still work? Are the matehd deprecated or removed?

Does this change the behavior in a breaking way for any CLI/APi users of Tasks.

@EhabY
Copy link
Contributor Author

EhabY commented Mar 3, 2026

The /experimental routes still exists so they will still work, this just registers pause/resume to /v2, there's even this comment:

// Tasks have been promoted to stable, but we have guaranteed a single
// release transition period where these routes must remain. These should
// be removed no earlier than Coder v2.30.0

So no breaking changes, just using the new stable API that we should've

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

X Tutup