Skip to content

Show personal details even if they are not updatable #5275

Closed

Description

@MorrisJobke

With the introduction of the global scale there is following configuration:

'gs.enabled' => true,
'gs.federation' => 'internal',

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.

cc @schiessle @oparoz @nickvergessen

Activity

  1. nickvergessen commented on Jun 7, 2017

    @nickvergessen
    Member

    Fine by me to display them if they are set.

  2. schiessle commented on Jun 7, 2017

    @schiessle
    Member

    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.

  3. schiessle commented on Jun 7, 2017

    @schiessle
    Member

    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?

  4. nickvergessen commented on Jun 7, 2017

    @nickvergessen
    Member

    @schiessle we don't have to sync anything from the lookup server. Just show the local data in the personal page?!

  5. blizzz commented on Jun 7, 2017

    @blizzz
    Member

    What @nickvergessen says :)

  6. schiessle commented on Jun 7, 2017

    @schiessle
    Member

    @nickvergessen yes, as said in my second comment... If we have them locally already through the user management it is fine

  7. nickvergessen commented on Jun 7, 2017

    @nickvergessen
    Member

    they can be set via provisioning with an admin account. So we just display them from the accounts table. 👍

  8. MorrisJobke commented on Jun 7, 2017

    @MorrisJobke
    MemberAuthor

    they can be set via provisioning with an admin account. So we just display them from the accounts table. 👍

    Exactly - that's what we are doing and that would solve this issue.

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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions