-
Notifications
You must be signed in to change notification settings - Fork 172
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
Remove List appendage for repeated fields #225
Comments
+1 |
@Jaypov did you figure out a way around this? I am having same problem here. |
This is a temporary solution for me. (<any>res).data |
Can't use repeated fields without hacks right now. I'm trying to use this library to interact with GRPC server and stuck on this issue. In my opinion this issue should be marked as important. Hope this will be fixed soon. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This is still an issue. |
Can someone provide the full description of the toolchain that is being used in this scenario please? The reason for asking is that the |
So this is not something that is caused by this plugin. It is the limitation of the official protoc compiler itself. Since this plugin has to match that scheme, it adds a suffix after repeated fields. I and a few other people have been maintaining this plugin which generates a typescript file directly instead of dealing with javascript and d.ts bindings. See: thesayyn/protoc-gen-ts#39 and https://github.com/thesayyn/protoc-gen-ts |
I have the same issue. |
The appendage of "List" to repeated fields causses issues, can this be changed / is there a way to stop it generating with the appendage for the following reason.
Example message
Generated output
Expected output
The Issue
No data returned over GRPC
Data successfully returned over GRPC
Service interface
Response on calling server
For a use case where you do not want to use instances of the generated classes, but instead just plain objects, the typings are rendered useless. any chance this can be changed or is there hacky way to resolve this?
The text was updated successfully, but these errors were encountered: