smnbbrv / ngx-grpc

Angular gRPC framework
MIT License
238 stars 34 forks source link

fix(protoc-gen-ng): properties are always generated in optional interfaces #97

Closed damirsaifut closed 1 year ago

damirsaifut commented 1 year ago

Values are not optional.

  export interface AsObject {
    message?: string;
    shouldThrow?: boolean;
    timestamp?: googleProtobuf000.Timestamp.AsObject;
  }

proto3

message EchoRequest {
  string message = 1;
  bool shouldThrow = 2;
  google.protobuf.Timestamp timestamp = 3;
}

This PR fixes it. Questions, suggestions and remarks are welcomed.

smnbbrv commented 1 year ago

Thank you very much!

Большое Спасибо!

damirsaifut commented 1 year ago

Thank you very much!

Большое Спасибо!

Not at all.

Не за что !

smnbbrv commented 1 year ago

Проверь 3.1.1, там wkt поправлены

damirsaifut commented 1 year ago

Проверь 3.1.1, там wkt поправлены

Обновить пакеты ? Я посмотрел уже код. Если не против, я залью мр еще ? У меня есть идея по реализации кода интерцептора и хэндлера.

smnbbrv commented 1 year ago

Да пожалуйста, вопрос только в том есть ли нужда в этом и генерализуемо ли, а так - конечно

damirsaifut commented 1 year ago

Да пожалуйста, вопрос только в том есть ли нужда в этом и генерализуемо ли, а так - конечно

В целом генерализуемо, но особой нужды нет, но улучшению кода нет предела. Я считаю, что можно развивать эти библиотеки, так grpc набирает обороты, а нормальных библиотек мало. У вас вышло хорошо как по мне