Skip to content
This repository was archived by the owner on Jul 16, 2025. It is now read-only.

Conversation

@OskarStark
Copy link
Contributor

@OskarStark OskarStark commented Jan 6, 2025

I am unsure about the logic, as this structure is only working for OpenAPI:
CleanShot 2025-01-06 at 13 53 44@2x

Any ideas @chr-hertel ?

@OskarStark OskarStark self-assigned this Jan 6, 2025
@OskarStark OskarStark added the enhancement New feature or request label Jan 6, 2025
@OskarStark OskarStark requested a review from chr-hertel January 6, 2025 12:54
@chr-hertel
Copy link
Member

hmm, maybe this is a good argument to go down the road of $response->toArray(false) in the ResponseConverter instead of throwing ClientException?

@OskarStark
Copy link
Contributor Author

Yes why not

@OskarStark
Copy link
Contributor Author

I pushed an update, is this what you had in mind? Code untested right now

@chr-hertel chr-hertel merged commit bbd06dc into main Jan 13, 2025
7 checks passed
@chr-hertel chr-hertel deleted the feature/exception branch January 13, 2025 12:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants