Skip to content

Conversation

@JamesNK
Copy link
Member

@JamesNK JamesNK commented Sep 6, 2022

More changes for consistency with other gRPC swagger generators

  • Add default error response to gRPC transcoding.
  • Use status instead of error.
  • Use name instead of full name for service name.

@JamesNK JamesNK added the area-grpc Includes: GRPC wire-up, templates label Sep 6, 2022
@JamesNK JamesNK added this to the 7.0-rc2 milestone Sep 6, 2022
@Pilchie
Copy link
Member

Pilchie commented Sep 7, 2022

Approved for .NET 7 RC2.

@JamesNK
Copy link
Member Author

JamesNK commented Sep 7, 2022

@dotnet/aspnet-build Please merge.

@wtgodbe wtgodbe merged commit 9952c6b into release/7.0 Sep 7, 2022
@wtgodbe wtgodbe deleted the jamesnk/transcoding-swagger-defaultresponse branch September 7, 2022 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-grpc Includes: GRPC wire-up, templates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants