search

fsr-de / pretix-wallet

Other
3 stars 0 forks source link

Display full name instead of email #9

Closed jeriox closed 6 months ago

jeriox commented 6 months ago 
          We can get a full name from the KeyCloak, don't we wanna use that here?

image

Originally posted by @dasGoogle in https://github.com/fsr-de/pretix-wallet/issues/1#issuecomment-1994661751

jeriox commented 6 months ago

@dasGoogle for me this was emtpy when testing locally, but maybe I just misconfigured something in my keycloak

dasGoogle commented 6 months ago

I'll check again, but I am relatively certain that this did indeed work as this was where I noticed that names were not updated (or for that matter, set in the first place) after the account is created. I used the myhpi testing realm in our keycloak (edited the allowed redirect URLs for the myHPI client there as well) and set the corresponding OIDC property in prefix to "name", if you want to try it yourself.

jeriox commented 6 months ago

jup works

jeriox commented 6 months ago

changed in https://github.com/fsr-de/pretix-wallet/commit/19815b4ae4a20abbfc331895ac8ff8a93f5242ce