Philip Gichuhi

Results 103 comments of Philip Gichuhi

@baywet the constructors... PHP 8 promotes constructor arguments to class properties. Ordinary constructors would still work ```php class UsersRequestBuilderGetRequestConfiguration { public function __construct( public ?array $headers = null, public ?array...

Regarding increase in size, I would argue it's a cost we can pay to make the developer experience better in this case. However, I'll measure the size impact of having...

You're right actually. No need to do the assignments. We wouldn't need to declare properties and doc comments would still be available as far as I know. ```php /** *...

Closing since there's no feedback from customer.

@Deadpikle @ZalemCitizen @Adstokoe19 @dhaggman are you able to share the curl version you're running? _PHP sample_ ```php curl_version()["version"]; curl_version()["features"]; ``` Might be a bug in my checks for whether is...

Version `1.87.0` contains a bug fix on the checks before enabling HTTP/2 on Guzzle. Please let me know if this resolves the issue

No further correspondence from customers. Please re-open if this is not fixed

Closed in last week's generation - https://github.com/microsoftgraph/msgraph-sdk-php/pull/1197

The v2.0 docs provide a way to do this: https://github.com/microsoftgraph/msgraph-sdk-php#create-a-token-request-context Please re-open in case the sample is not clear.

This has been resolved in v2.0 of this SDK. Kindly consider upgrading.