search

rogchap / wombat

Cross platform gRPC client
MIT License
1.4k stars 52 forks source link

Syntax error when using google.protobuf.Timestamp #13

Closed zpatrick closed 3 years ago

zpatrick commented 3 years ago

I'm getting a syntax error when trying to specify a google.protobuf.Timestamp. This seems to occur no matter what the input for either startTime or nanos is:

image

rogchap commented 3 years ago

Hmmm, interesting; thanks for reporting; will hopefully get a fix for this soon.

rogchap commented 3 years ago

Ok; I can see the issue: some/most of the Well Known Types have custom marshal/unmarshal functions that I need to account for.

I always planned on better support for Well Known Types, I guess this bug will expedite that!

I think I'll do a quick fix for Timestamp and Duration and plan in the rest later.

zpatrick commented 3 years ago

Awesome, thanks for the quick reply!

rogchap commented 3 years ago

v0.2.2 has support for Timestamp and Duration.... In the future I'd like to add a Date/Time picker for Timestamp to make it easier to input.

rogchap commented 3 years ago

Looking at your screenshot again @zpatrick I think you are going to have similar issues with the Wrapper types too: BoolValue 😞

Will add it to my list.

zpatrick commented 3 years ago

ah bummer, sounds good man!

rogchap commented 3 years ago

FYI v0.3.x has support for the wrapper types

oheyzhiwei commented 2 years ago

Just encountered this when dealing with FieldMask https://developers.google.com/protocol-buffers/docs/reference/csharp/class/google/protobuf/well-known-types/field-mask