You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Show personal details even if they are not updatable #5275
With this all inputs fields for the lookup server (phone number, website, address, twitter) on the personal page are hidden. We should maybe show them without the input field but only as text if they are set. In this way the user sees what data is associated to it's account if it is set by the admin somehow.
hm, the problem is that at the moment we only query the lookup server once you want to share with someone but we don't check the lookup server regularly for changes of the user data and sync them back to the Nextcloud server.
While thinking about it. The email address is normally also configured in the user management and we copy if from there already, right? So we don't need to query the lookup server to get at least this value.
I'm quite sure that this is true for LDAP, right @blizzz @MorrisJobke would this also work in your case?
Activity
nickvergessen commentedon Jun 7, 2017
Fine by me to display them if they are set.
schiessle commentedon Jun 7, 2017
hm, the problem is that at the moment we only query the lookup server once you want to share with someone but we don't check the lookup server regularly for changes of the user data and sync them back to the Nextcloud server.
schiessle commentedon Jun 7, 2017
While thinking about it. The email address is normally also configured in the user management and we copy if from there already, right? So we don't need to query the lookup server to get at least this value.
I'm quite sure that this is true for LDAP, right @blizzz
@MorrisJobke would this also work in your case?
nickvergessen commentedon Jun 7, 2017
@schiessle we don't have to sync anything from the lookup server. Just show the local data in the personal page?!
blizzz commentedon Jun 7, 2017
What @nickvergessen says :)
schiessle commentedon Jun 7, 2017
@nickvergessen yes, as said in my second comment... If we have them locally already through the user management it is fine
nickvergessen commentedon Jun 7, 2017
they can be set via provisioning with an admin account. So we just display them from the accounts table. 👍
MorrisJobke commentedon Jun 7, 2017
Exactly - that's what we are doing and that would solve this issue.