souramoo/commentoplusplus

Guest names even if anonymous authentication is disabled

Aperta

#56 aperta il 28 lug 2021

Vedi su GitHubGuida contributor
 (2 commenti) (1 reazione) (0 assegnatari)Go (67 fork)user submission
enhancementgood first issue

Metriche repository

Star
 (433 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Currently, you can only provide guest names when anonymous authentication is enabled, but I think it would be a great feature if you would have an option to make the name mandatory when writing a comment.

commento_anonymously_enabled

As it is now, you can still click the "COMMENT ANONYMOUSLY" button and therefore don't have to enter a name, but if you disable the "anonymous authentication" feature you have no name field at all.

commento_anonymously_disabled

Let me know what you think of the idea and we can further discuss the implementation detail then.

Guida contributor

Direzione di ricerca
Inizia esaminando il codice corrente di gestione dell'autenticazione, probabilmente nel backend, per capire come vengono gestiti i commenti anonimi. Quindi, esplora i template frontend che visualizzano il modulo di commento e il pulsante 'Commenta in modo anonimo'. Cerca eventuali opzioni di configurazione esistenti che controllano se il campo del nome viene mostrato. L'obiettivo è aggiungere una nuova impostazione che imponga l'inserimento di un nome anche quando l'autenticazione anonima è disabilitata, e modificare la logica di invio per richiedere un nome quando tale impostazione è abilitata.
Tech stack
go
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65