Skip to content

Prefer explicit project flag over environment vars - #98

Merged
davidzhao merged 1 commit into
mainfrom
pass-project
Sep 27, 2022
Merged

davidzhao merged 1 commit into
mainfrom
pass-project

Conversation

@davidzhao

Copy link
Copy Markdown
Member

No description provided.

@davidzhao
davidzhao merged commit 51e8f54 into main Sep 27, 2022
@davidzhao
davidzhao deleted the pass-project branch September 27, 2022 05:56
Topherhindman added a commit that referenced this pull request Oct 23, 2026
The Public API is dropping WorkspaceService.UpdateProject
(livekit/public-api-server#98). It duplicated ProjectService.UpdateProject,
and its PATCH path collided with the workspace GetProject/DeleteProject path
in the OpenAPI spec.

`lk workspace project update` keeps its interface. UpdateWorkspaceProject now
reads the project through the workspace first, so a project outside
--workspace still answers 404, and then renames it with
ProjectService.UpdateProject. This works against the server before and after
#98, and the next client regen no longer needs WorkspaceServiceUpdateProject.
Topherhindman added a commit that referenced this pull request Oct 23, 2026
Picks up livekit/public-api-server#98, which makes the spec a client
contract. Every operation now declares its error response, a
google.rpc.Status, so each generated response gains a JSONDefault
*GoogleRpcStatus, and GoogleRpcStatus and GoogleProtobufAny are new
types. Nothing hand-written reads them yet.

#98 also deletes WorkspaceService.UpdateProject, so its client methods
and types are gone. lk workspace project update no longer calls it
(#1071).

Also picks up the server's proto comment cleanup, which roughly halves
the doc comments, and the oneOf helpers of a few protocol messages
(AutoTrackEgress, the egress outputs, chat and transcript items, metric
points) losing their empty last variant. Nothing hand-written uses
those either.

Generated from livekit/public-api-server@4cfd5cf
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.

2 participants