nextcloud/server

Option to disable showing system email in avatars' contacts menu

Open

#23 172 ouverte le 4 oct. 2020

Voir sur GitHub
 (2 commentaires) (6 réactions) (0 assignés)PHP (4 865 forks)batch import
1. to developenhancementfeature: contacts menugood first issueprivacy

Métriques du dépôt

Stars
 (34 953 stars)
Métriques de merge PR
 (Merge moyen 20j) (627 PRs mergées en 30 j)

Description

Steps

  1. Setup a NC instance
  2. Create accounts for many external people with email so they can receive an invitation and notifications
  3. Create a Talk conversation with all people
  4. Click on the three dots on a person's avatar in the participant list

Expected result

In the case of external people, we should not see their email address.

Actual result

There's an option to see and click an email address

I suggest to add an option to remove the EmailProvider in config.php or other. A quick hack is to remove https://github.com/nextcloud/server/blob/stable20/lib/private/Contacts/ContactsMenu/ActionProviderStore.php#L88 in config.php

Version

Nextcloud 20

@rullzer @ChristophWurst FYI

Guide contributeur

Direction de recherche
Ajoutez une nouvelle option dans config.php pour désactiver le EmailProvider dans le menu des contacts. Référez vous à ActionProviderStore.php et modifiez le pour vérifier l'option de configuration avant d'enregistrer le fournisseur.
Stack technique
php
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PHPNextcloud development setup
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65