AspNetCore.Docs icon indicating copy to clipboard operation
AspNetCore.Docs copied to clipboard

.NET 7.0 Update: gRPC JSON transcoding: add support for multi-segment parameters

Open wadepickett opened this issue 2 years ago • 4 comments

In the 7.0 version of the gRPC JSON transcoding add a new section: Support for multi-segment parameters which was a feature added in .NET 7.0 preview 6.

This issue has a parent tracking issue: #26477


Document Details

Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

wadepickett avatar Aug 04 '22 00:08 wadepickett

@JamesNK , should I add a new section to this doc: Support for multi-segment parameters, reflecting what you provided in 7.0 preview 6 blog, or do you envision that being covered elsewhere?

wadepickett avatar Aug 04 '22 00:08 wadepickett

There is a link to https://cloud.google.com/service-infrastructure/docs/service-management/reference/rpc/google.api#google.api.HttpRule from our doc which covers how to annotate the proto files.

I would eventually like to summarize annotating on our gRPC JSON transcoding doc.

This particular feature - multi-segment parameters - is unlikely to be used by many people. Don't need to talk about it.

JamesNK avatar Aug 04 '22 02:08 JamesNK

@JamesNK Thanks! It looks like that is what you already covered annotating the .proto here correct?
Announcing gRPC JSON transcoding for .NET.

OK for me to get a PR started using your work from that blog and examples as a guide, or is this something you want to do yourself? If you do want me to start a PR, do you want me to expand at all on Would you want me to expand at all on HttpRule options? (I will leave out multi-segment parameters).

wadepickett avatar Aug 04 '22 17:08 wadepickett

@JamesNK, Sorry for the needless noise, cancel that request on annotating. I realized simply covering it as a step as it is in this doc already (of course) and you were just wanting to summarize it better. I'll give that a shot given what you already have available with a PR for you to review.

wadepickett avatar Aug 10 '22 00:08 wadepickett

Tracking the new summary for annotation in #26688. Closing this issue on multi-segment since we don't need it.

wadepickett avatar Aug 17 '22 17:08 wadepickett