Artur Souza

Results 337 comments of Artur Souza

We should create a new API that is HTTP specific instead of having a break change here. This API was originally designed to be protocol agnostic.

> > We should create a new API that is HTTP specific instead of having a break change here. This API was originally designed to be protocol agnostic. > >...

Thanks. It is because the dashboard has a different version itself. Thanks for bringing this up.

We had this problem in the CLI recently. Updating the CLI version used in the dashboard might fix this.

Please, do not extend ObjectSerializer, it is used for internal objects only. Simply, implement DaprObjectSerializer interface. The PR should focus on handling the content-type value correctly and not extending DefaultObjectSerializer.

@abogdanov37 This fix must be undone because it is a breaking change. Data stored with the old version cannot be read with this patch. We are reverting this and undoing...

@abogdanov37 For this feature to be added, the old way should be the default and the new way must be opted in. The design would need to be a little...

@abogdanov37 Like we discussed in the issue, I will look into that before merging the PR. Ideally, I can come up with a new unit tests that can gate this...

ActorReference is a class that can be serialized by user provided serialization and it might not be compatible with every serializer, specially when cross SDK compatibility is required. I propose...

This is a missing feature. It was never designed to run inside a container for local mode, only K8s. I can take a PR that enabled container mode for the...