-
Notifications
You must be signed in to change notification settings - Fork 3
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
No found message or enum for google.protobuf.Duration or google.protobuf.Timestamp #12
Comments
@cayden-uw Can you show an example of a proto file? Did you add this file to the project? google/protobuf/duration.proto
Our library generates clean code to control what needs to be turned on. Perhaps there is a need to add |
Hi, I've made a minimal reproduction that shows the issue. syntax = "proto3";
import "google/protobuf/duration.proto";
import "google/protobuf/timestamp.proto";
message Sample {
string value_a = 1;
google.protobuf.Timestamp value_b = 2;
google.protobuf.Duration value_c = 3;
} When I run it
If I comment out Is there some other way to import packages or something? PS, thanks for the lib 😊 |
Found a workaround. From the same sample message definition. First download the Timestamp and Duration definitions and put them in the same folder.
Then run it.
This gives output with an index.js like.
Ideally, the library would know how to resolve these dependencies like the other proto generators. Thanks |
When running
npx @whisklabs/grpc
on my proto file I get errors for every duration and timestamp field like the error belowI compared my proto to the one in your tests which uses timestamps and couldn't find any differences in imports or the one the fields are defined.
I am using Ubuntu 21.10, I have protoc installed and can successfully generate js libaries using protoc grpc-web
The text was updated successfully, but these errors were encountered: