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.

Sign up for GitHub

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

Jump to bottom

Adopt http\routes from cadl ranch #5101

Closed
jorgerangel-msft opened this issue Nov 14, 2024 · 1 comment
Closed

Adopt http\routes from cadl ranch #5101

jorgerangel-msft opened this issue Nov 14, 2024 · 1 comment
Assignees
@JoshLove-msft
Labels
emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp

Comments

@jorgerangel-msft
Copy link
Contributor

Adopt https://github.com/Azure/cadl-ranch/tree/main/packages/cadl-ranch-specs/http/routes
@jorgerangel-msft jorgerangel-msft added the emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp label Nov 14, 2024
@JoshLove-msft JoshLove-msft self-assigned this Nov 15, 2024
@JoshLove-msft
Copy link
Contributor

JoshLove-msft commented Nov 15, 2024 •
edited
Loading

There is a bug in the cadl-ranch scenarios Azure/cadl-ranch#772

Also, it appears our emitter (or TCGC) is not setting Explode correctly #5134

The emitter or TCGC is including query parameters in the Path when the query parameter is part of the route - #5139

@JoshLove-msft JoshLove-msft mentioned this issue Nov 15, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue Nov 18, 2024
@JoshLove-msft
Adopt routes Cadl-ranch spec (#5138)
b003714 
Contributes to #5101
@JoshLove-msft JoshLove-msft removed their assignment Nov 18, 2024
@JoshLove-msft JoshLove-msft self-assigned this Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JoshLove-msft JoshLove-msft

Labels
emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JoshLove-msft @jorgerangel-msft