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

gRPC examples: Why are .proto in impl and not api? #145

Open
ryanhanks opened this issue Nov 23, 2019 · 2 comments
Open

gRPC examples: Why are .proto in impl and not api? #145

ryanhanks opened this issue Nov 23, 2019 · 2 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed type:improvement

Comments

@ryanhanks
Copy link

Do the samples currently demonstrate best practices around where .proto files should live?

The examples currently don't share a schema definition between the client and the implementing service. Why is that? Is this not something that we would expect to see in the api package, and then used by both the service making the client call and the service providing the handler?

@octonato
Copy link
Member

You are right about it. This should be added to the api module. I think this was just an oversight.

Thanks for reporting this.

I added this issue to the backlog, but it can take sometime before someone can find time to pick it.

Would you be interested in sending a PR?

@octonato octonato added good first issue Good for newcomers help wanted Extra attention is needed type:improvement labels Nov 28, 2019
@ignasi35
Copy link
Contributor

Actually, I don't think that's supported at the moment. I thought we had a specific inssue in either lagom/lagom or playframework/play-grpc` (which includes lagom support for grpc) but I could only find akka/akka-grpc#152 (and related issues).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed type:improvement
Development

No branches or pull requests

3 participants