Gary PEGEOT

Results 16 comments of Gary PEGEOT

@derrabus I switched to Attribute, way nicer indeed, thanks. Not sure about the optional validation though, for two reasons: 1. It imply to create custom normalization for `PartialDenormalizationException` instead of...

> > Would it make sense to split this into 2 different features: one that deserialize input and one that validate controller arguments (with 2 separate attributes)? > > This...

@derrabus @jvasseur so one way to do it would be to listen for `ControllerArgumentsEvent` and check by reflexion if the controller has a specific attribute (Something like `#[Validate(['foo', 'bar'])]` ?),...

@lyrixx @dunglas would make sense to create an `AbstractProblemNormalizer` to regroup `ConstraintViolationListNormalizer` and the new `PartialDenormalizationProblemNormalizer`?

@soyuka in my case, the class doesn't have any `id` property (identifier is named `sku`). I've added `#[ApiProperty(identifier: true)]` on top of my property, but for some reason identifiers contains...

> > > Hello, > Thank you for this PR, sorry for the delay in reviewing it. > Does it support merging rules with already existing acces_control rules? We should...

> A rebase is required Rebase done !

Hi @jmariller ! Are you still working on this? Maybe I can help on the HttpClientTransport issue

Hi @damienalexandre! Correct me if I'm wrong, but it looks like `JoliCode\Elastically\Transport\HttpClientTransport` will not have much sense with ruflin's Elastica v8: 1. AbstractTransport is gone 2. `Elastic\Transport\Transport` is a final...

I'm creating a PR on damien's fork to remove the transport while adding test to make sure you can specify your own PSR Compliant client if you want to. This...