bitwarden/server

"Create Account" Button should be inactive/hidden when "globalSettings__disableUserRegistration=true"

Open

#1 268 ouverte le 11 avr. 2021

Voir sur GitHub
 (11 commentaires) (6 réactions) (1 assigné)C# (1 575 forks)batch import
enhancementgood first issue

Métriques du dépôt

Stars
 (18 588 stars)
Métriques de merge PR
 (Merge moyen 11j 10h) (147 PRs mergées en 30 j)

Description

Hi everyone,

The topic is not entirely new and an attempt has already been made to address it with issue #631 . Unfortunately the task was closed.

But I would still be of the opinion that it would make sense to deactivate the "Create Account" button and hide it as soon as the config option globalSettings__disableUserRegistration is set to "true".

At the moment it is possible to enter all data including email, username and password and only on submitting the register form user gets an error toast. This is somehow unnecessary and also arouses false expectations among users

Guide contributeur

Direction de recherche
Étudiez comment l'option de configuration 'globalSettings disableUserRegistration' est utilisée dans le code du serveur. Recherchez la vue ou le contrôleur d'inscription qui affiche le bouton 'Créer un compte'. Consultez la tentative précédente #631 pour le contexte. Le changement implique probablement de masquer conditionnellement le bouton dans la vue en fonction de la valeur de configuration.
Stack technique
csharphtmlcss
Domaine
frontendauthentication
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.
3
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.
Bloquée
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#HTMLCSS
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60