feat(segments): add SegmentUpdateAsync for PATCH /segments/:id - #153
Conversation
There was a problem hiding this comment.
No issues found across 8 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Requires human review: Adds new public SegmentUpdateAsync API for PATCH /segments/:id plus CLI command and fake server; needs human sign-off because it expands the public API surface and is a new feature, not a bounded fix.
Re-trigger cubic
There was a problem hiding this comment.
0 issues found across 3 files (changes from recent commits).
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Requires human review: Adds new public SegmentUpdateAsync API for PATCH /segments/:id plus CLI command and fake server; expands public API surface as a new feature, so needs human sign-off.
Re-trigger cubic
| @@ -0,0 +1,42 @@ | |||
| using McMaster.Extensions.CommandLineUtils; | |||
There was a problem hiding this comment.
let's maybe not update the CLI? we've already deprecated it anyway, we don't need to update it
Adds
SegmentUpdateAsyncforPATCH /segments/:id, which renames a segment.Params
name(string, required) — the new segment name.Response
{ "object": "segment", "id": "...", }