Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix PHP request executor methods returning enums #2854

Merged
merged 1 commit into from
Jul 6, 2023

Conversation

Ndiritu
Copy link
Contributor

@Ndiritu Ndiritu commented Jul 5, 2023

Calls sendPrimitiveAsync on the request adapter with the enum class name.

depends on microsoft/kiota-http-guzzle-php#69
closes microsoftgraph/msgraph-beta-sdk-php#182

@Ndiritu Ndiritu added the PHP label Jul 5, 2023
@Ndiritu Ndiritu self-assigned this Jul 5, 2023
@Ndiritu Ndiritu requested a review from a team as a code owner July 5, 2023 14:46
@Ndiritu Ndiritu marked this pull request as draft July 5, 2023 14:53
@Ndiritu Ndiritu force-pushed the fix/php-requestexecutor-enum branch from b92a5d1 to 38f7d69 Compare July 5, 2023 14:54
@Ndiritu Ndiritu marked this pull request as ready for review July 5, 2023 15:02
@Ndiritu Ndiritu marked this pull request as draft July 5, 2023 15:07
@Ndiritu
Copy link
Contributor Author

Ndiritu commented Jul 5, 2023

Fixing integration test failure

@Ndiritu Ndiritu force-pushed the fix/php-requestexecutor-enum branch from 38f7d69 to dd70b2d Compare July 6, 2023 09:21
@Ndiritu
Copy link
Contributor Author

Ndiritu commented Jul 6, 2023

Fixing integration test failure

Will be addressed in separate PR. Issues are not related to this change & seem to be occurring as a result of duplicate properties in composed type wrappers and some models, not just for PHP.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

92.9% 92.9% Coverage
0.0% 0.0% Duplication

@Ndiritu Ndiritu marked this pull request as ready for review July 6, 2023 09:37
Copy link
Member

@andrueastman andrueastman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏼

@Ndiritu Ndiritu merged commit b6ee9fb into main Jul 6, 2023
@Ndiritu Ndiritu deleted the fix/php-requestexecutor-enum branch July 6, 2023 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix deserialization of enums
3 participants