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

Added openid.connect.userInfo #143

Closed
wants to merge 3 commits into from

Conversation

nemanja-ivanis
Copy link

Added openid.connect.userInfo

Added openid
Added openid.connect.userInfo endpoint
updated name
@pyrech
Copy link
Member

pyrech commented Mar 7, 2022

Hi @nemanja-ivanis,

Thanks a lot for your contribution. You successfully updated the patched json spec of the API but you now need to regenerate the patch and the PHP SDK. We have some documentation here in case you need help 😉

@chaseconey
Copy link

@pyrech @nemanja-ivanis Is this still in the works? The issue is pretty old, but this functionality seems still missing.

If it's just a matter of regenerating something, I can give it a go if there is no bandwidth to do it.

@pyrech
Copy link
Member

pyrech commented Apr 12, 2023

Hello @chaseconey, yes this PR should probably need only to regenerate the SDK from the updated spec. We would happily accept a PR to fix that. Thanks 🙂

@lyrixx
Copy link
Member

lyrixx commented Apr 19, 2023

Hello,

I'm closing this PR for lack of activity.

If someone want this PR to be merged, don't hesitate to open a new PR. Don't forget to read this comment

Thanks for the initial work 👍🏼

@lyrixx lyrixx closed this Apr 19, 2023
@chaseconey
Copy link

I tried to pull down the latest spec and it seems to be erroring out. Is that a known issue? Happy to take this and run with it otherwise.

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.

None yet

4 participants