Skip to content

Implement JsonSerializable so OpenApi instance can be serialized using json_encode() #183

Open
grimmdude wants to merge 8 commits intocebe:masterfrom
appsumo:json_serialize
Open

Implement JsonSerializable so OpenApi instance can be serialized using json_encode() #183
grimmdude wants to merge 8 commits intocebe:masterfrom
appsumo:json_serialize

Conversation

@grimmdude
Copy link

Allows the ability to serialize OpenApi instance like this:

$openApi = Reader::readFromYaml($yaml);
$json = json_encode($openApi);

@cebe
Copy link
Owner

cebe commented Apr 23, 2025

Looks useful, but I'd add some tests before merging.

@cebe cebe added this to the 1.9.0 milestone Apr 23, 2025
@grimmdude
Copy link
Author

Looks useful, but I'd add some tests before merging.

Hi @cebe, I'll leave the tests to you. It just calls the existing getSerializableData() method when using json_encode().

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants