Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adopt http\server\path\multiple from cadl ranch #3986

Closed
m-nash opened this issue Jul 25, 2024 · 3 comments · Fixed by #4691
Closed

Adopt http\server\path\multiple from cadl ranch #3986

m-nash opened this issue Jul 25, 2024 · 3 comments · Fixed by #4691
Assignees
Labels
emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp

Comments

@m-nash
Copy link
Member

m-nash commented Jul 25, 2024

Adopt https://github.com/Azure/cadl-ranch/tree/main/packages/cadl-ranch-specs/http/server/path/multiple

@m-nash m-nash added the emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp label Jul 25, 2024
@chunyu3 chunyu3 self-assigned this Sep 29, 2024
@chunyu3
Copy link
Contributor

chunyu3 commented Sep 29, 2024

depend on #4206

@chunyu3
Copy link
Contributor

chunyu3 commented Oct 10, 2024

Need to implement #4673

@chunyu3
Copy link
Contributor

chunyu3 commented Oct 11, 2024

Need to resolve #4690

github-merge-queue bot pushed a commit that referenced this issue Oct 17, 2024
Fix #3986
Fix #4673
Fix #4690

- support api-version is a path parameter. We will add a `_apiVersion`
field in the client, and will use `_apiVersion` field in
`CreateXXXMessage`
- When the type of api-version is Enum, we will convert it to string
- Not to generate constructor for `ClientOptionsProvider` in
`stubLibraryVistor`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants