ccouzens / keycloak-openapi

OpenAPI definitions for Keycloak's Admin API
168 stars 66 forks source link

fix: some of the response types #36

Closed kallberg closed 11 months ago

kallberg commented 11 months ago

Hello.

Noticed the conversation on the pull-request for version 22 and decided to give it a try. I realized later when I looked at the network graph that you were already in the process of updating using version-22 branch.

I rebased and squashed commits to match your banch.

Don't know what the state of your local branches are so maybe there is a lot of duplicate work and if that is the case please feel free to ignore/close this PR.

Otherwise here's some changes that should fix some of the response type parsing for v22. I don't know if I got all of them.

tried to get most of the types fixed probably missed some

also:

ccouzens commented 11 months ago

Hey - sorry I saw this too late.

As you may have noticed from me taking months to get to version 22 I don't currently have much time for this. Would you be interested in being a co-maintainer on this project? You'd be doing me a favour.

Chris

kallberg commented 11 months ago

Hi Chris,

I'm interested. At least for the next couple of months as the the specs made available thanks to this project will come in handy for me soon. Was considering maintaining a fork.

Erik

ccouzens commented 11 months ago

Hey Erik,

You should now have access.

kallberg commented 11 months ago

Got it!

I'll send you an email tomorrow with some basic questions about how you'd like us to collaborate